Skip to content

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

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

Merged
brendancol merged 1 commit into from
May 18, 2026
+8 −0
Merged

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

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .claude/commands/rockout.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,12 @@ plotting conventions, GIS alert boxes, preview images, and humanizer passes.
```
5. Capture the PR number for the next step.

**Do NOT wait for CI to finish before moving on to Step 9.** Push the PR
and proceed to the review immediately. CI runs asynchronously and the
review-pr / follow-up loop runs in parallel. If CI surfaces a failure
later, address it as a separate follow-up commit on the same branch --
do not block the review pass on green CI.

## Step 9 -- Run the Domain-Aware PR Review

1. Invoke the `/review-pr` command against the PR number from Step 8:
Expand All @@ -161,6 +167,8 @@ plotting conventions, GIS alert boxes, preview images, and humanizer passes.
- **Blockers** -- must fix before merge
- **Suggestions** -- should fix, not blocking
- **Nits** -- optional improvements
4. Run this step regardless of CI status. Do not poll `gh pr checks` or
wait for workflows to finish before invoking `/review-pr`.

## Step 10 -- Follow Up on Review Findings

Expand Down
Loading