sync: pull upstream changes from HackYourFuture/data-assignment-week-5#1
Merged
Conversation
Bring the c55 cohort assignment scaffold in sync with upstream main: - Renumber tasks to match the 8-task chapter (adds "Task 3: Write Tests"; shifts subsequent task numbers by 1 across Dockerfile, ci.yml, pipeline.py, AI_ASSIST.md, .hyf/test.sh, and README). - Update README to scope `az login` to the HYF tenant and link the Notion assignment page instead of the dead GH Pages URL. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
lassebenni
commented
Jun 1, 2026
lassebenni
left a comment
lassebenni
left a comment
Contributor
Author
There was a problem hiding this comment.
Local review by Claude - no issues found.
- Diff matches HackYourFuture/data-assignment-week-5@main byte-for-byte (git diff origin/sync/upstream-main..upstream/main is empty).
- Repo CI (.hyf/test.sh autograder, 9s) passed on the head SHA.
- No vt-dbt / PBIR / powerbi validators in the review-pr workflow apply to this curriculum scaffold repo.
- No new code paths, credentials, or conflict markers introduced.
- The one em-dash in AI_ASSIST.md is pre-existing on both sides of the diff (and exists upstream) - not flagged.
(Cannot self-APPROVE since I authored the PR; submitting as COMMENT.)
Contributor
Author
|
@copilot review |
I reviewed the current PR state and there are no actionable issues for me to change in code right now. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Syncs this cohort repo with the canonical upstream at HackYourFuture/data-assignment-week-5.
Dockerfile,.github/workflows/ci.yml,src/pipeline.py,AI_ASSIST.md,.hyf/test.sh, andREADME.md.az loginto the HYF tenant in the README and replaces the dead GH Pages link with the Notion assignment page.Test plan
README.mdmatch the labels inDockerfile,ci.yml,pipeline.py,AI_ASSIST.md, and.hyf/test.sh.bash .hyf/test.sh) against an unchanged solution.🤖 Generated with Claude Code