tanjilahmed87@gmail.com

Engineering Culture5 min read

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.