Skip to content

Grant contents: read to the eval gate workflow and docs example#145

Merged
fkapsahili merged 1 commit into
masterfrom
fix-eval-gate-contents-permission
Jun 14, 2026
Merged

Grant contents: read to the eval gate workflow and docs example#145
fkapsahili merged 1 commit into
masterfrom
fix-eval-gate-contents-permission

Conversation

@fkapsahili

Copy link
Copy Markdown
Collaborator

The eval-gate workflow and its documented example declare permissions: pull-requests: write. Specifying a permissions: block sets every unlisted scope to none, including contents, so actions/checkout runs without contents: read. On a private repository that returns fatal: repository '...' not found (a 404 for the unauthorized token), and the gate never runs.

It works in this public repo because a public checkout does not require the token's contents scope, which is why it went unnoticed. Teams running the gate on a private repository (the common case for an internal eval gate) hit the failure as soon as they copy the documented workflow.

This grants contents: read alongside pull-requests: write in both our own eval-gate.yml and the regression-gate.md example, so the gate checks out on public and private repositories alike.

@fkapsahili
fkapsahili merged commit 95a6550 into master Jun 14, 2026
9 checks passed
@fkapsahili
fkapsahili deleted the fix-eval-gate-contents-permission branch June 14, 2026 14:08
@github-actions

Copy link
Copy Markdown
Contributor

✅ Eval gate passed

metric value
pass rate 100% → 100% (0 pts)
significant false
regressed cases 0
improved cases 0
added / removed 0 / 0
pairing positional

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