-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Closed
Labels
bugSomething isn't workingSomething isn't workingcoreThis issue is not accepting PRs from outside contributorsThis issue is not accepting PRs from outside contributorsp2Affects more than a few users but doesn't prevent core functionsAffects more than a few users but doesn't prevent core functions
Description
Describe the bug
The pr checks --watch command doesn't list and wait for all checks when there are 2 sets (e.g. one on "push" and another one on "pull_request") - it only waits for 1 set. I think this used to work correctly when --watch was first implemented (I am the author of the original feature request for that flag).
❯ gh --version
gh version 2.30.0 (2023-05-30)
https://github.com/cli/cli/releases/tag/v2.30.0
Steps to reproduce the behavior
- Have a repo with CI set to
on: push: branches: "*" pull_request:
- Create a PR
- Have the "push" checks finish before the "pull_request" checks
- Run
gh pr checks --watch - The CLI reports all checks as finished, but the "pull_request" ones aren't - e.g. can't do
gh pr merge(if the checks are required)
Expected vs actual behavior
Expected: The command will wait for all check to finish
Actual: The command only waits for one set of checks to finish
Logs
Example PR: skaut/skaut-google-drive-gallery#2037
❯ gh pr checks --watch
All checks were successful
0 failing, 6 successful, 0 skipped, and 0 pending checks
✓ Build 2m8s https://github.com/skaut/skaut-google-drive-gallery/actions/runs/5325655157/jobs/9646625675
✓ Lint 2m26s https://github.com/skaut/skaut-google-drive-gallery/actions/runs/5325655004/jobs/9646752460
✓ PHP 5.6 compati... 14s https://github.com/skaut/skaut-google-drive-gallery/actions/runs/5325655157/jobs/9646625445
✓ Test 2m4s https://github.com/skaut/skaut-google-drive-gallery/actions/runs/5325655004/jobs/9646752718
✓ codecov/patch https://app.codecov.io/gh/skaut/skaut-google-drive-gallery/pull/2037
✓ codecov/project https://app.codecov.io/gh/skaut/skaut-google-drive-gallery/pull/2037
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingcoreThis issue is not accepting PRs from outside contributorsThis issue is not accepting PRs from outside contributorsp2Affects more than a few users but doesn't prevent core functionsAffects more than a few users but doesn't prevent core functions