Grey.ooo/agent-standards
Watch
2
0
Fork
You've already forked agent-standards
 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

Watch PRs to mergeable, not just green #15

Merged
geusebio merged 1 commit from docs/pr-mergeable-alongside-ci into main 2026-08-09 22:58:48 +00:00
Conversation 0 Commits 1 Files changed 2 +35 -14
geusebio commented 2026-08-09 22:56:33 +00:00
Owner
Copy link

Extends the "watch CI to green" habit so it also covers merge state, not just check status.

  • CI passing and mergeable are independent: main moving underneath the branch mid-run, a
    newly-required review, or a branch-protection rule can all block merge on a fully green run.
  • When watching a PR for a check to resolve, watch its merge state alongside it
    (tea pulls <index> --fields state,ci,mergeable), and keep watching both until they hold at
    the same time — re-check mergeable after CI resolves, not only before.
  • A regression in either (a new conflict, a requested review, a check going red again) means
    the work isn't done, not a blip to wave off.

Updates CLAUDE.md's "Seeing changes through" habit and VCS.md's "Pull requests" / "Work in
progress" sections + quick reference, reconciled against the actionlint/tea-0.14.2 content
already on main.

Extends the "watch CI to green" habit so it also covers merge state, not just check status. - CI passing and mergeable are independent: `main` moving underneath the branch mid-run, a newly-required review, or a branch-protection rule can all block merge on a fully green run. - When watching a PR for a check to resolve, watch its merge state alongside it (`tea pulls <index> --fields state,ci,mergeable`), and keep watching both until they hold at the same time — re-check mergeable after CI resolves, not only before. - A regression in either (a new conflict, a requested review, a check going red again) means the work isn't done, not a blip to wave off. Updates `CLAUDE.md`'s "Seeing changes through" habit and `VCS.md`'s "Pull requests" / "Work in progress" sections + quick reference, reconciled against the actionlint/tea-0.14.2 content already on `main`.
CI passing and mergeable are independent: main moving underneath a branch, a newly-required review, or a branch-protection rule can all block merge on a fully green run. Watch mergeable state alongside CI, re-checking it after CI resolves, and treat a regression in either as still in progress.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Grey.ooo/agent-standards!15
No description provided.