ci: gate CI for external contributors behind copy-pr-bot approval#1399
Open
kajalj22 wants to merge 1 commit into
Open
ci: gate CI for external contributors behind copy-pr-bot approval#1399kajalj22 wants to merge 1 commit into
kajalj22 wants to merge 1 commit into
Conversation
External contributor fork PRs now require a maintainer to approve via copy-pr-bot before unit tests and linting run. Matches the NeMo-RL pattern where CI only runs on `pull-request/NNN` branches. Changes: - Set copy-pr-bot `auto_sync_ready: false` (require manual vetter approval) - Add `push` trigger on `pull-request/[0-9]+` branches to unit-tests and code-linting workflows (CI path for approved external PRs) - Add concurrency groups to cancel stale runs - Add cleanup workflow to delete `pull-request/NNN` branches on PR close Internal contributors pushing directly to the repo are unaffected — the `pull_request` trigger remains for non-fork PRs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Kajal Jain <kajalj@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
auto_sync_ready: falseso fork PRs require manual vetter approval via/ok to test <sha>before CI runspushtrigger onpull-request/[0-9]+branches to unit-tests and code-linting workflowspull-request/NNNbranches when PRs closepull_requesttrigger remainsHow it works
/ok to test <commit-sha>→ copy-pr-bot createspull-request/NNNbranch → unit tests and linting trigger viapushpull-request/NNNbranchWhat changed
.github/copy-pr-bot.yamlauto_sync_ready: false.github/workflows/unit-tests.ymlpushtrigger forpull-request/[0-9]+, concurrency group, fix draft PR check for push events.github/workflows/code-linting.ymlpushtrigger forpull-request/[0-9]+, concurrency group.github/workflows/clean-copy-pr-bot-branch.ymlpull-request/NNNbranch on PR closeTest plan
pull_request/ok to test <sha>on the fork PR and confirm copy-pr-bot createspull-request/NNNbranchpull-request/NNNpushpull-request/NNNbranch is deleted🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com