Skip to content

Require approval to run CI on draft PRs#4789

Merged
raydouglass merged 1 commit intorapidsai:branch-25.02from
bdice:copy-pr-bot-disable-drafts
Nov 26, 2024
Merged

Require approval to run CI on draft PRs#4789
raydouglass merged 1 commit intorapidsai:branch-25.02from
bdice:copy-pr-bot-disable-drafts

Conversation

@bdice
Copy link
Contributor

@bdice bdice commented Nov 26, 2024

By default, CI runs on draft PRs. This leads to many CI runs that may be unnecessary.

With this PR's change to .github/copy-pr-bot.yaml, an /ok to test comment from a trusted user is required to trigger CI on draft PRs. Non-draft PRs will run CI by default, assuming that all commits are signed by trusted users. Otherwise an /ok to test is required (as before) -- see the copy-pr-bot docs at https://docs.gha-runners.nvidia.com/apps/copy-pr-bot/ for more information.

Part of rapidsai/build-planning#123.

@bdice bdice requested a review from a team as a code owner November 26, 2024 18:59
@bdice bdice requested a review from msarahan November 26, 2024 18:59
@bdice bdice added non-breaking Non-breaking change improvement Improvement / enhancement to an existing function labels Nov 26, 2024
@jameslamb jameslamb removed the request for review from msarahan November 26, 2024 20:04
@jameslamb
Copy link
Member

The docs-build failure here is unrelated to this PR's changes.

ref: #4784 (comment)

@raydouglass raydouglass merged commit e57fc5c into rapidsai:branch-25.02 Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants