The code review comment that changed how our teams work
Four words in a pull request template did more for our review culture than any process document: "What made this hard?"
Engineering Practice
Tvisha Engineering
Our pull request template used to ask the usual things: what changed, how to test it, which ticket. Reviews were slow, comments skewed towards formatting, and the genuinely risky parts of a change routinely went unexamined because nobody knew where they were.
We added one field: "What made this hard?" It is optional. It changed everything about the reviews.
Why it works
The author is the only person who knows which twenty lines of a four-hundred-line diff kept them up. Asking them to name it hands the reviewer a map. Reviews got shorter and better at the same time, because attention went where the risk was instead of being spread evenly across a diff.
It also does something subtler. Admitting difficulty in writing normalises it. Junior engineers stopped performing effortlessness, which meant they started asking for help earlier — the single largest source of avoidable delay on any team we have measured.
The latency problem
Most teams believe their review problem is depth. Measure it and you usually find it is latency: a pull request that waits eleven hours for a first comment costs far more than one reviewed shallowly in twenty minutes, because the author has context-switched and must reload everything to respond. We now track time-to-first-comment as a team health metric and leave review depth to the engineers.
What we do not do
We do not mandate a minimum comment count, block merges on approval from a specific senior, or run style discussions in review — the formatter owns style and it is not negotiable, which removes an entire genre of argument. A review is a conversation about risk between two people who both want the change to ship.
Keep reading
Onboarding engineers into regulated environments in under two weeks
Access approvals, compliance training and a codebase nobody has documented. How we get placed engineers productive without cutting a single control.
6 min readRetrofitting AI into legacy enterprise systems without a rewrite
Most enterprises do not need a greenfield platform to benefit from machine learning. A pragmatic pattern for wrapping models around the systems you cannot replace.
8 min readStaff augmentation or a dedicated development centre? A decision framework
Duration, domain depth and IP sensitivity decide this — not headcount. The four questions we ask every client before recommending a model.
6 min read