Repository navigation
Add allow-fork input to let one named fork publish - #51
Conversation
Forks such as DeterminateSystems/nix-eval-jobs skip publishing since c74b8bc fixed the fork guard. The allow-fork input names the one repository that can publish even though it is a fork. Forks of that repository still skip publishing. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe workflow adds an optional ChangesFork publishing
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to The new fork path requires a nonempty repository-name match and retains the existing publishing gates; no merge-blocking regression is established. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
c74b8bc fixed the fork guard on the publish steps.
Before that fix, the guard never blocked anything.
Now repositories that are GitHub forks skip publishing without an error, and the job still reports success.
DeterminateSystems/nix-eval-jobs is a fork of NixOS/nix-eval-jobs.
Its releases after v3.21.9 did not reach FlakeHub because of this.
For example, in run 32404936425 (v3.22.1) the checkout, Nix, cache and flakehub-push steps all show "skipped".
This PR adds an
allow-forkinput.Set it to the
owner/reponame of the fork that is allowed to publish.Publishing happens only when
github.repositorymatches that value exactly, so forks of that fork still skip publishing.When
allow-forkis empty, the behavior does not change.actionlint, prettier and zizmor pass locally.
🤖 Generated with Claude Code
Summary by CodeRabbit
allow-forkoption, including the requiredowner/repoformat and how it determines whether a fork can publish.