File tree Expand file tree Collapse file tree 1 file changed +18
-0
lines changed
Expand file tree Collapse file tree 1 file changed +18
-0
lines changed Original file line number Diff line number Diff line change 1+ ## Review Philosophy
2+
3+ - Only comment when you have HIGH CONFIDENCE (>80%) that an issue exists
4+ - Be concise: one sentence per comment when possible
5+ - Focus on actionable feedback, not observations
6+ - When reviewing text, only comment on clarity issues if the text is genuinely confusing or could lead to errors.
7+
8+ ## Skip These (Low Value)
9+
10+ Do not comment on:
11+
12+ - ** Style/formatting** - CI handles this (rustfmt, prettier)
13+ - ** Test failures** - CI handles this (full test suite)
14+ - ** Missing dependencies** - CI handles this (mypy will fail)
15+ - ** Minor naming suggestions** - unless truly confusing
16+ - ** Suggestions to add comments** - for self-documenting code
17+ - ** Multiple issues in one comment** - choose the single most critical issue
18+ - ** Pedantic accuracy in text** - unless it would cause actual confusion or errors. No one likes a reply guy
You can’t perform that action at this time.
0 commit comments