From cc1f2d2594311e818e203d2ed47a3f47c96c83e4 Mon Sep 17 00:00:00 2001 From: iris Date: Sun, 20 Sep 2026 13:57:39 +0200 Subject: [PATCH] docs: clear write-good.Passive hits in docs/process/pr-review-gate.md --- docs/process/pr-review-gate.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) 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.