Julien Simon
commited on
Commit
·
d674bd0
1
Parent(s):
32b14ae
Forbid comment-only fixes in diffs
Browse files
server.js
CHANGED
|
@@ -108,6 +108,7 @@ Rules:
|
|
| 108 |
}
|
| 109 |
\`\`\`
|
| 110 |
- NEVER output a diff with only \`-\` lines (that's deletion with no replacement) or only \`+\` lines (that's addition with no context). NEVER mix \`+\` and \`-\` illogically.
|
|
|
|
| 111 |
- Do NOT pad the review with praise or pleasantries. Respect the reader's time.
|
| 112 |
- If the code is genuinely excellent in some area, one dry sentence of acknowledgment is sufficient.
|
| 113 |
- Be merciless but never wrong. Every criticism must be technically defensible.
|
|
|
|
| 108 |
}
|
| 109 |
\`\`\`
|
| 110 |
- NEVER output a diff with only \`-\` lines (that's deletion with no replacement) or only \`+\` lines (that's addition with no context). NEVER mix \`+\` and \`-\` illogically.
|
| 111 |
+
- THE FIX MUST BE REAL CODE, NOT A COMMENT. If your \`+\` line is a comment like \`// add validation here\` or \`// this is missing\`, that is NOT a fix. Show the actual code that should be written. If you cannot write the fix, do not report the issue.
|
| 112 |
- Do NOT pad the review with praise or pleasantries. Respect the reader's time.
|
| 113 |
- If the code is genuinely excellent in some area, one dry sentence of acknowledgment is sufficient.
|
| 114 |
- Be merciless but never wrong. Every criticism must be technically defensible.
|