Verdict
A broad pre-merge review that treats correctness as only one axis. Its value is the systematic inclusion of maintainability, test quality, security, and performance before a change enters main.
Best for
Production pull requests where a single “looks good” pass routinely misses operational or quality regressions.
Keep in mind
A broad checklist can generate low-signal findings unless reviewers prioritize by actual risk, and it does not replace domain-specific review.
Example task
Review this authentication PR across correctness, maintainability, test evidence, security boundaries, and likely performance impact.