diff --git a/docs/process/pr-review-gate.md b/docs/process/pr-review-gate.md index 132a28fb..d8fd1960 100644 --- a/docs/process/pr-review-gate.md +++ b/docs/process/pr-review-gate.md @@ -46,8 +46,9 @@ already happened. No large changes are expected to surface after that point — a reviewer's job past that point is to flag it if one does, not to assume none ever will. -The practical consequence for a reviewer: on a repo where automerge -may already be armed before your review lands, a plain `APPROVED` can +The practical consequence for a reviewer: on a repo where someone with +write access may already have armed automerge before your review +lands, a plain `APPROVED` can be the last step before the merge actually happens, with no further review pass after yours. That's a reason to actually finish checking before approving — not a reason to hesitate over every small thing.