Pull Request Review Checklist
When to use
Run before approving any pull request for merge.
Checklist
- Tests cover the change — MUST pass.
- No secrets or credentials in the diff — MUST pass.
- Public API changes are documented — MUST pass.
- CI is green — MUST pass.
Rules
- NEVER approve with failing CI.
- ALWAYS block on a missing test for new behavior.
Example
A PR adds a new endpoint but no tests. Block on the missing-test item and request coverage before approving.