Skip to content

ci: run PR body check on fork PRs via pull_request_target - #4

Merged
lassebenni merged 1 commit into
mainfrom
fix/pr-body-check-pull-request-target
Jul 28, 2026
Merged

ci: run PR body check on fork PRs via pull_request_target#4
lassebenni merged 1 commit into
mainfrom
fix/pr-body-check-pull-request-target

Conversation

@lassebenni

Copy link
Copy Markdown
Contributor

Summary

  • PR body check was not firing on student fork PRs (e.g. Halyna R. #1) with pull_request only.
  • Also trigger on pull_request_target so fork PRs run the check from main. Job still only reads the PR body (no head checkout).

How to review

  • Diff is only .github/workflows/pr-body-check.yml.
  • After merge, edit Halyna R. #1's description and confirm the body check runs (and fails until the template headings are present).

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.

Made with Cursor

Fork student PRs were not picking up the pull_request-only body check. Also listen on pull_request_target (body-only, no head checkout) so the check runs from main for HackYourAssignment forks.

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 49feacc into main Jul 28, 2026
1 of 2 checks passed
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