Skip to content

rockout: skip CI wait before review-pr and follow-ups#2035

Merged
brendancol merged 1 commit into
mainfrom
rockout-skip-ci-wait
May 18, 2026
Merged

rockout: skip CI wait before review-pr and follow-ups#2035
brendancol merged 1 commit into
mainfrom
rockout-skip-ci-wait

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

  • Adds an explicit instruction after Step 8 (Open the Pull Request) telling the rockout workflow not to wait for CI before moving to Step 9.
  • Adds a matching item in Step 9 forbidding polling of `gh pr checks` or workflow completion before invoking `/review-pr`.
  • CI runs asynchronously and the review / follow-up loop happens in parallel. A CI failure becomes a separate follow-up commit on the same branch.

Test plan

  • Trigger a rockout run on a small change and confirm `/review-pr` is invoked without waiting for CI.
  • Confirm a later CI failure is handled as a follow-up commit on the same branch rather than blocking the review loop.

Adds an explicit note after Step 8 and an item in Step 9 telling the
workflow not to poll CI before running /review-pr or before applying
suggestion/nit fixes. CI surfaces failures asynchronously; the review
loop runs in parallel and any CI failure becomes a separate follow-up
commit on the same branch.
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label May 18, 2026
@brendancol brendancol merged commit dc7534a into main May 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant