Code Review Roadmap¶

• Roadmap: https://roadmap.sh/code-review

Deep-Dive Sections¶

• 01. Review Etiquette — tone, blameless framing, suggesting alternatives, when to defer
• 02. Asking Questions — questions over commands, Socratic review, surfacing assumptions
• 03. Blocking vs Nit — distinguishing must-fix from preference, prefix conventions
• 04. Async Review — time-zone-friendly review, draft PRs, review SLAs
• 05. When to Approve — coverage of concerns, scope creep, follow-up TODOs

Companion materials: - Clean Code / PR Review Checklist — design-level review checklist - Clean Code / Pre-Commit Checklist — what the author should do before requesting review

1. Code Styles¶

• 1.1 Smaller effort for changes later on
• 1.2 Automate here

2. Tests¶

• 2.1 Smaller effort for changes later on
• 2.2 Automate here

3. Documentation¶

• 3.1 Focus on these
• 3.2 Higher effort for changes later on

4. Implementation Semantics¶

• 4.1 Focus on these
• 4.2 Higher effort for changes later on

5. API Semantics¶

• 5.1 Focus on these
• 5.2 Higher effort for changes later on