Building a Code Review Culture That Catches Bugs, Not Egos
The difference between a code review culture that improves a codebase and one that just creates friction is almost entirely about how feedback is phrased.
Tanjil Ahmed
Lead Software Engineer · Notionhive
Code review is where engineering culture is most visible, because it's the place disagreement happens in writing, attached to someone's name, on a regular cadence. A team's actual values show up here faster than in any mission statement.
- Comment on the code, never the coder — 'this loop reruns the query' not 'you forgot to think about performance again.'
- Distinguish blocking issues from preference with a clear label ("nit:") so reviewers stop gatekeeping on style.
- Approve with comments when the issue is real but not urgent — don't hold a working PR hostage to a preference.
- Reviewers ask questions before asserting — 'what happens if this list is empty?' invites explanation instead of assuming a mistake.
The teams with the healthiest review culture I've worked on treat a review as a conversation about the code, held in public, where being wrong about a suggestion costs nothing. That psychological safety is what makes engineers actually flag the subtle bug instead of staying quiet to avoid a confrontation.
A code review culture that punishes being wrong teaches engineers to stop pointing out what's wrong.
