chadtmiller.com/knotless
git that lives in your Atmosphere account
1
10
AGENTS.md says to dismiss a gate that stopped reviewing, not force past it
AGENTS.md+9−2
| @@ -251,8 +251,15 @@ name review.knotless.dev (`knotless rules`), waits for its review of the | ||
pull's head: `knotless show` says `waiting for its review` meanwhile. | ||
Ejected with `changes asked by @review.knotless.dev`, read the review | ||
with `knotless show`, push the fix, and queue it again: the agent reads | ||
the new head. Where the review is wrong, say why in a comment and ask | ||
the owner, whose approval of the head stands in for it. An approving | ||
the new head. Ejected for `not reviewed as main asks` when it approved | ||
an earlier head and has stopped reviewing (it reviews a pull `--rounds` | ||
times), and nothing since is more than `main` merged in or its own notes | ||
answered, dismiss its review, saying so (`knotless dismiss <repo> <pull | ||
uri> review.knotless.dev <reason>`), and queue it again: a dismissal | ||
stands in for a gate's review, on the owner's own pull too, where an | ||
approval never counts. Where it asked for changes and is wrong, say why | ||
in a comment and ask the owner whether to dismiss it. Never `--force` | ||
past it: that skips the checks at the head too. An approving | ||
review can still carry notes it does not hold the pull for: once the | ||
pull lands, read its review with `knotless show` and judge each note, | ||
checking first that it is right, as a reviewer can be wrong. Fix it in | ||