Skip to content

fix: align scaffold env vars and links with Week 13 curriculum - #4

Merged
lassebenni merged 3 commits into
mainfrom
fix/align-scaffold-with-curriculum
Jul 28, 2026
Merged

fix: align scaffold env vars and links with Week 13 curriculum#4
lassebenni merged 3 commits into
mainfrom
fix/align-scaffold-with-curriculum

Conversation

@lassebenni

@lassebenni lassebenni commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Rename DBRICKS_*DATABRICKS_* in .env.example and profiles.yml.example so they match Ch4 / the assignment chapter.
  • Fix Task 1 README link (week_13__6_assignment.mdweek_13__7_assignment.md).
  • Document --full-refresh baseline in Task 2 README; mention optional task-4/ in root README layout.
  • Add PR body check enforcing the Week 13 PR template headings.

How to review

  • Diff is small: env var rename, two README wording fixes, optional task-4/ line, and .github/workflows/pr-body-check.yml.
  • Confirm Grade Assignment workflow is unchanged from the fork-safe pin already on main.

Secrets hygiene checklist

  • No .env or profiles.yml files committed.
  • No Databricks personal access tokens (dapi...) hardcoded.
  • profiles.yml.example and .env.example templates present.

Test plan

  • Confirm .env.example / profiles.yml.example use DATABRICKS_*
  • Confirm Task 1 README opens week_13__7_assignment.md
  • bash .hyf/test.sh still fails the empty scaffold (pass=false)
  • PR body check + Grade Assignment both report on this PR after merge to main (body check runs from PR head)

Rename DBRICKS_* to DATABRICKS_* to match Ch4, point Task 1 at week_13__7_assignment.md, and document the --full-refresh baseline plus optional task-4 layout.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

This comment has been minimized.

Enforce Summary / How to review / Secrets hygiene checklist so student PRs cannot skip the PR template.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

This comment has been minimized.

Match the cohort fix so student fork PRs reliably get the body-check workflow from main.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 10 / 100 ❌

Status: ❌ Not passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Databricks Lab Autograder ===
  ✗ FAIL  task-1: add pyspark_exploration.ipynb or .py
  ✗ FAIL  task-2/dbt_project.yml missing
  ✓ PASS  task-2/WRITEUP.md
  ✓ PASS  task-3/SCHEDULING.md
  ✓ PASS  AI_ASSIST.md
  ✓ PASS  Level 1: required files (0/15 pts)
  ✓ PASS  Level 2: PySpark notebook (0/25 pts)
  ✗ FAIL  models/marts/fct_trips.sql (or similar) not found
  ✗ FAIL  WRITEUP should explain is_incremental()
  ✗ FAIL  WRITEUP should reference {{ this }} or the incremental filter
  ✓ PASS  Level 3: dbt incremental + WRITEUP (0/30 pts)
  ✗ FAIL  task-3/screenshots/ should contain screenshots of Job config, green run, and paused schedule
  ⚠ WARN  SCHEDULING.md missing Databricks Job Run URL (paste URL from address bar)
  ✗ FAIL  SCHEDULING.md should explain when to use Databricks Jobs vs Airflow
  ✓ PASS  Level 4: Task 3 Job Scheduling (0/15 pts)
  ⚠ WARN  AI_ASSIST.md needs your documented prompt and rationale
  ✓ PASS  .gitignore excludes .env and profiles.yml
  ✓ PASS  profiles.yml.example present
  ✓ PASS  Level 5: AI_ASSIST & secrets hygiene (10/15 pts)

Score: 10 / 100  (passing: 60)  pass=false
Total: 10/100 — pass=false (passing threshold: 60)

@lassebenni
lassebenni merged commit 8bac2b6 into main Jul 28, 2026
1 of 2 checks passed
@lassebenni
lassebenni deleted the fix/align-scaffold-with-curriculum branch July 28, 2026 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant