Skip to content

fix(ci): grant contents: read so the grade workflow loads#5

Merged
lassebenni merged 1 commit into
mainfrom
fix/grade-workflow-permissions
Jun 2, 2026
Merged

fix(ci): grant contents: read so the grade workflow loads#5
lassebenni merged 1 commit into
mainfrom
fix/grade-workflow-permissions

Conversation

@lassebenni
Copy link
Copy Markdown
Collaborator

Summary

The Grade Assignment workflow currently fails to load on every PR with:

The nested job test is requesting contents: read, but is only allowed contents: none.

The reusable workflow at HackYourFuture/github-actions/.../auto-grade.yml@main has a nested job that requires contents: read to checkout the PR code. The caller here only granted issues + pull-requests, which caps every other permission at none. Adding contents: read to the caller's permissions block unblocks the nested checkout step with no other behaviour change.

Test plan

  • Merge this PR.
  • Open any test PR and confirm the Grade Assignment workflow loads and runs (rather than failing at workflow-parse time).

🤖 Generated with Claude Code

The reusable workflow at HackYourFuture/github-actions/.github/workflows/auto-grade.yml@main
contains a nested job that requests 'contents: read' (to checkout the
PR code). The caller job here only granted issues + pull-requests,
which caps every other permission at 'none'. Result: every PR fails
to load the grade workflow with

  The nested job 'test' is requesting 'contents: read', but is only
  allowed 'contents: none'.

Adding 'contents: read' to the caller's permissions block unblocks the
nested checkout step. No other behaviour changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@lassebenni lassebenni merged commit 79426e8 into main Jun 2, 2026
1 check passed
@lassebenni lassebenni deleted the fix/grade-workflow-permissions branch June 2, 2026 14:38
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