Skip to content

Ship dbt_project.yml; drop dbt init from Task 1#4

Merged
lassebenni merged 1 commit into
mainfrom
remove-dbt-init
Jul 5, 2026
Merged

Ship dbt_project.yml; drop dbt init from Task 1#4
lassebenni merged 1 commit into
mainfrom
remove-dbt-init

Conversation

@lassebenni

Copy link
Copy Markdown
Collaborator

Situation

The Week 10 assignment template shipped every support file except dbt_project.yml. Task 1 asked students to run dbt init in a throwaway subfolder, move the generated dbt_project.yml up, delete the rest, then hand-edit it to strip the example: block.

Problem

That step was the #1 autograder footgun: any leftover models/example/my_first_dbt_model.sql makes dbt build report FAIL 1 not_null_my_first_dbt_model_id and blocks the ERROR=0 requirement. It also taught a distorted version of the real workflow (dbt init normally scaffolds a project in place, not a delete-the-rest dance).

Solution

Ship a ready-to-run dbt_project.yml so students fill in stubs instead of scaffolding.

  • Add dbt_project.ymlstaging = view, marts = table, profile: nyc_taxi_borough_daily (matches profiles.yml.example).
  • README — Task 1 is now "connect to the shared database" (profiles.yml + dbt debug); file tree no longer marks dbt_project.yml as NOT included.
  • dbt-parse CIdbt_project.yml now always ships, so parse always runs. Verified dbt deps && dbt parse is green against the raw TODO stubs.

Paired with curriculum edits in the datatrack repo (assignment Task 1, Ch2 forward-reference, rubric, lesson plan).

The template is now a ready-to-run dbt project. Students no longer run
`dbt init` and clean up its example scaffolding (the #1 autograder
footgun: leftover models/example -> FAIL not_null_my_first_dbt_model_id).

- add dbt_project.yml (staging = view, marts = table), matching the
  profiles.yml.example profile name.
- README: Task 1 is now "connect to the shared database"; file tree
  no longer marks dbt_project.yml as "NOT included".
- dbt-parse CI: dbt_project.yml now always ships, so parse always runs.
  Verified `dbt deps && dbt parse` is green against the raw stubs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lassebenni lassebenni merged commit bd36cac into main Jul 5, 2026
1 check passed
@lassebenni lassebenni deleted the remove-dbt-init branch July 5, 2026 21:58
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