Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Disable CI on draft PRs #123

Closed
bdice opened this issue Nov 26, 2024 · 1 comment
Closed

Disable CI on draft PRs #123

bdice opened this issue Nov 26, 2024 · 1 comment

Comments

@bdice
Copy link
Contributor

bdice commented Nov 26, 2024

Recently @KyleFromNVIDIA added a feature to copy-pr-bot that allows opting in to skipping CI on draft PRs: https://github.com/nv-gha-runners/roadmap/issues/157

See https://docs.gha-runners.nvidia.com/apps/copy-pr-bot/#configuration for configuration details.

I spoke with a handful of folks (@vyasr, @KyleFromNVIDIA, @raydouglass) and we would like to opt into auto_sync_draft: false on all RAPIDS repositories. This issue tracks that work.
This was referenced Nov 26, 2024
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
rapids-bot bot pushed a commit to rapidsai/cucim that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#798)
1f708ef 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #798
rapids-bot bot pushed a commit to rapidsai/cudf that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#17450)
797a07b 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #17450
rapids-bot bot pushed a commit to rapidsai/cuspatial that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#1491)
ee70be4 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #1491
rapids-bot bot pushed a commit to rapidsai/cuvs that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#498)
ae6816c 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #498
rapids-bot bot pushed a commit to rapidsai/dask-cuda that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#1412)
6d1601f 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #1412
rapids-bot bot pushed a commit to rapidsai/kvikio that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#563)
3b90520 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #563
rapids-bot bot pushed a commit to rapidsai/nx-cugraph that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#52)
b8c4a7d 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Erik Welch (https://github.com/eriknw)
  - James Lamb (https://github.com/jameslamb)

URL: #52
rapids-bot bot pushed a commit to rapidsai/raft that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#2512)
adfd2f6 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)
  - Jake Awe (https://github.com/AyodeAwe)

URL: #2512
rapids-bot bot pushed a commit to rapidsai/rmm that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#1737)
a6a455d 
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.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #1737
raydouglass pushed a commit to rapidsai/cuml that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#6145)
ae9c1ea 
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.
raydouglass pushed a commit to rapidsai/cugraph that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#4789)
e57fc5c 
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.
raydouglass pushed a commit to rapidsai/cugraph-gnn that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#80)
01abe44 
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.
raydouglass pushed a commit to rapidsai/integration that referenced this issue Nov 26, 2024
@bdice
Require approval to run CI on draft PRs (#734)
d43e43e 
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
Copy link
Contributor Author

bdice commented Nov 26, 2024

All the PRs are now merged (except for CCCL, which is not a RAPIDS repo -- and it is likely to merge soon). Closing.

@bdice bdice closed this as completed Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bdice