Requesting Code Review Skill

When to Use

• Code is complete and ready for review
• Before merging a significant change
• When you want feedback on implementation

When NOT to Use

• When the self-review checklist has not been completed — don't request external review for work you haven't reviewed yourself
• When tests are failing — fix them first; reviewers shouldn't spend time on code with a broken test suite
• For WIP / draft PRs where the implementation is known to be incomplete
• When only asking for a rubber stamp on something already merged — review happens before merge, not after

Pre-Review Checklist

Before requesting review, ensure: