Blog

Communication in Code Review: Comment Tone, Conventional Comments, Disagreement and Review Fatigue

A good review comment addresses the code rather than the person, gives its reason and states clearly whether it blocks the merge; Conventional Comments does this with labels such as issue, suggestion, question and praise, plus blocking and non-blocking decorations. The post rewrites the comments on the coupon pull request in that format and covers how many nits are too many, the author's reply, how to handle disagreement and review fatigue.