docs: clear write-good.Passive hits in docs/process/pr-review-gate.md
This commit is contained in:
parent
457cae708c
commit
cc1f2d2594
1 changed files with 3 additions and 2 deletions
|
|
@ -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.
|
||||
|
|
|
|||
Loading…
Reference in a new issue