Skip to content

Commit efcc783

Browse files
bloveclaude
andcommitted
docs(contributing): make AI-review-comment handling an explicit convention
The advisory AI review never blocks merge, so its comments can be silently inherited or ignored. Codify the team norm: before arming auto-merge, the author reads each AI comment and either addresses it in a follow-up commit or replies on the thread with the reason for deferring. A green/red review check says nothing about whether the comments were considered. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 2e69985 commit efcc783

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

CONTRIBUTING.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,12 @@ not a required check and never blocks a merge. A second workflow
4747
Scorecard's Code-Review check reads from the reviews API. The maintainer still
4848
merges every PR.
4949

50+
Because the review is advisory, **handling its comments is a convention, not a
51+
gate**: before arming auto-merge, the author reads each AI comment and either
52+
addresses it in a follow-up commit or replies on the thread with the reason for
53+
deferring/declining. Don't merge past unread review comments — the check going
54+
green (or red) says nothing about whether the comments were considered.
55+
5056
This credits Code-Review via automation rather than peer review, because the
5157
project is currently single-maintainer. OSSF documentation suggests
5258
automated/AI reviews may not be intended to count toward this check; the current

0 commit comments

Comments
 (0)