- Read the diff and the tests that cover it.
- Name what the change does, in one sentence.
- List defects that would ship a wrong result, a secret, or a broken install. Skip style nits.
- Say which checks ran and which did not.
- Leave the tree unchanged unless the asker asked for fixes.
About this skill
Review a diff