ci: re-pin PR Hygiene so its runs queue instead of cancelling - #4774
Conversation
A concurrency group holds one pending run and cancels it when the next event arrives, so a burst of events on one pull request cancelled the middle ones and a cancelled run renders as a failed check. Runs now queue in arrival order. dashpay/stale_prs_are_bad#26 and #27 landed as b378d1d7f9c3949a5f73822362ca02d251ab8141. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🕓 Review not started yet because the new head is waiting for the 30-minute push debounce.
Commit 287a7df. Normal review starts when eligible; priority review starts as soon as a slot is available. |
Re-pins the shared review workflow onto dashpay/stale_prs_are_bad#26 and #27.
Runs queue instead of cancelling. A concurrency group holds one pending run and cancels it when the next event arrives, so three events on one pull request inside a run's duration cancelled the middle one — and a cancelled run renders as a failed check. That was 1494 of 6556 runs over five days, and it is why pull requests have been going red.
concurrency.queue: maxlets them wait in arrival order instead; nothing is cancelled, and runs for one pull request stay serialised.One duplicated comment no longer freezes an author's queue. Two runs reconciling the same pull request at once could each open a controller state comment, after which every later run refused to read that pull request's history — and the handler for that refusal puts an
errorstatus on every open pull request of that author. Hourly, until someone deleted a comment by hand. The oldest record now wins; nothing is deleted.Also: a refused label or reviewer request is reported and skipped rather than aborting the run.
Verified on dash-evo-tool before this round:
concurrency.queueis accepted in a called workflow and the run succeeds.No change to who may approve, when a pull request is ready, or what blocks it. The only line that differs here is the pinned engine revision.
🤖 Generated with Claude Code