Skip to content

feat(task-3): replace Azure proof screenshot with Blob Storage upload#2

Merged
lassebenni merged 3 commits intomainfrom
feat/task3-blob-upload
May 7, 2026
Merged

feat(task-3): replace Azure proof screenshot with Blob Storage upload#2
lassebenni merged 3 commits intomainfrom
feat/task3-blob-upload

Conversation

@lassebenni
Copy link
Copy Markdown
Collaborator

Summary

  • Task 3 redesigned: students now upload their pipeline output (clean_sales.csv) to a private Azure Blob container via the portal's Storage Browser, instead of just screenshotting the resource group. This gives hands-on experience with the exact service they'll use programmatically in Week 5.
  • Grader updated (.hyf/test.sh): checks for task-3/assets/azure_blob_week2.png (10 pts) + task-3/assets/blob_url.txt containing a valid *.blob.core.windows.net URL (10 pts). Same 20-point total.
  • Scaffold added: task-3/assets/ folder replaces the bare .gitkeep.
  • README updated: Task 3 row and repo layout reflect new deliverables.

Prerequisite

Students need Storage Blob Data Contributor data-plane permissions on the shared storage account. The student-role.bicep custom role has been updated in the curriculum repo to add the required dataActions — redeploy before the cohort starts Week 2.

Test plan

  • Run .hyf/test.sh against a submission with only the screenshot → score shows 10/20 for Task 3
  • Run .hyf/test.sh against a submission with screenshot + valid blob URL → 20/20
  • Run .hyf/test.sh against a submission missing both → 0/20
  • Verify Task 1 and Task 2 scores are unchanged

🤖 Generated with Claude Code

Task 3 now asks students to upload their pipeline output (clean_sales.csv)
to a private Azure Blob container via the portal's Storage Browser, instead
of just screenshotting the resource group.

Changes:
- task-3/assets/ scaffold (screenshot + blob_url.txt placeholders)
- .hyf/test.sh: grader checks for azure_blob_week2.png (10pts) +
  blob_url.txt containing a valid *.blob.core.windows.net URL (10pts)
- README: updated Task 3 row and repo layout to match new deliverables

This comment was marked as outdated.

- Reword header comment to make clear the screenshot is required (10pts)
  and blob_url.txt earns the remaining 10pts only when screenshot exists
- Update echo label from 'Azure Proof' to 'Azure Blob Upload'
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

Comment thread README.md Outdated
Comment thread .hyf/test.sh Outdated
Comment thread .hyf/test.sh Outdated
…sions in README

- Require container/blob path in blob URL regex so a bare storage account
  root URL does not satisfy the check
- Split blob_url.txt check into missing vs present-but-invalid with distinct
  messages so students can diagnose failures accurately
- Add jpg/jpeg to Task 3 table cell in README to match grader behaviour
@lassebenni lassebenni merged commit 9a71f3d into main May 7, 2026
@lassebenni lassebenni deleted the feat/task3-blob-upload branch May 7, 2026 15:41
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.

2 participants