Skip to content

Baraah A.#7

Open
thebaraah wants to merge 6 commits into
HackYourAssignment:mainfrom
thebaraah:main
Open

Baraah A.#7
thebaraah wants to merge 6 commits into
HackYourAssignment:mainfrom
thebaraah:main

Conversation

@thebaraah

Copy link
Copy Markdown

No description provided.

thebaraah added 2 commits June 2, 2026 20:56
Sync from HackYourAssignment/c55-data-week-5 (Grade workflow fix)
@github-actions

This comment has been minimized.

GitHub withholds AZURE_CREDENTIALS from pull_request runs that originate
from a fork, causing the azure/login step to fail. The if-guard skips the
Azure/ACR steps on fork PRs so CI stays green. The central Grade ACR push
workflow on the assignment repo's main handles the actual push from
base-repo context (where the secret is available) and comments the image
URL on the PR.
@github-actions

This comment has been minimized.

@lassebenni lassebenni closed this Jun 3, 2026
@lassebenni lassebenni reopened this Jun 3, 2026
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

Image pushed to ACR

hyfregistry.azurecr.io/thebaraah-pipeline:a0b7aad94a26ed2f649c8bb2463b1141cf237e45

Browse the registry in the Azure Portal: https://portal.azure.com/#@hackyourfuture.nl/resource/subscriptions/1120c89d-2a5f-4a15-a582-2ea34f0bb5c3/resourceGroups/rg-hyf-data/providers/Microsoft.ContainerRegistry/registries/hyfregistry/repository

Task 7 verification: your Dockerfile built and pushed cleanly from commit a0b7aad.

@HackYourAssignment HackYourAssignment deleted a comment from github-actions Bot Jun 3, 2026
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

Image pushed to ACR

hyfregistry.azurecr.io/thebaraah-pipeline:33960ce696387ab3ed003ce547faa5e0ae882115

Browse the registry in the Azure Portal: https://portal.azure.com/#@hackyourfuture.nl/resource/subscriptions/1120c89d-2a5f-4a15-a582-2ea34f0bb5c3/resourceGroups/rg-hyf-data/providers/Microsoft.ContainerRegistry/registries/hyfregistry/repository

Task 7 verification: your Dockerfile built and pushed cleanly from commit 33960ce.

@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

Image pushed to ACR

hyfregistry.azurecr.io/thebaraah-pipeline:2d8a960adc2334abe2296caadf18ad0d0031fa5e

Browse the registry in the Azure Portal: https://portal.azure.com/#@hackyourfuture.nl/resource/subscriptions/1120c89d-2a5f-4a15-a582-2ea34f0bb5c3/resourceGroups/rg-hyf-data/providers/Microsoft.ContainerRegistry/registries/hyfregistry/repository

Task 7 verification: your Dockerfile built and pushed cleanly from commit 2d8a960.

@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 86 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Week 5 Autograder ===
  PASS: Level 1: required files (15/15 pts)
  PASS: Dockerfile uses python:3.11-slim base image
  FAIL: Dockerfile does not copy requirements before source code
  PASS: Dockerfile has a CMD instruction
  PASS: Level 2: Dockerfile (8/15 pts)
  PASS: tests/test_pipeline.py has 12 test functions (≥2 required)
  PASS: tests/test_pipeline.py has no NotImplementedError stubs remaining
  PASS: Level 3: unit tests (10/10 pts)
  PASS: requirements.txt has 9 pinned package(s)
  PASS: requirements.txt pins satisfied (no uv.lock needed)
  PASS: Level 4: pinned dependencies (10/10 pts)
  PASS: ci.yml triggers on pull_request
  PASS: ci.yml triggers on push to main
  PASS: ci.yml runs ruff check (lint)
  PASS: ci.yml runs ruff format (format check)
  PASS: ci.yml runs pytest
  PASS: ci.yml runs docker build
  PASS: Level 5: CI workflow (20/20 pts)
  PASS: pipeline.py reads config from os.environ/os.getenv
  FAIL: pipeline.py still contains NotImplementedError
  PASS: Level 6: env-var config (10/15 pts)
  PASS: assets/acr_push_week5.png present and non-trivial (321775 bytes)
  PASS: Level 7: ACR screenshot (10/10 pts)
  PASS: AI_ASSIST.md has all three required sections
  FAIL: AI_ASSIST.md still contains TODO placeholders or is too short (906 chars)
  PASS: Level 8: AI report (3/5 pts)
  PASS: .gitignore correctly excludes __pycache__/, *.pyc, and .env

Score: 86 / 100  (passing: 60)  pass=true

@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown

Image pushed to ACR

hyfregistry.azurecr.io/thebaraah-pipeline:d088982482bf6981d291a62dbf2271e82bb3555e

Browse the registry in the Azure Portal: https://portal.azure.com/#@hackyourfuture.nl/resource/subscriptions/1120c89d-2a5f-4a15-a582-2ea34f0bb5c3/resourceGroups/rg-hyf-data/providers/Microsoft.ContainerRegistry/registries/hyfregistry/repository

Task 7 verification: your Dockerfile built and pushed cleanly from commit d088982.

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