Code review is een van de activiteiten met het hoogste hefboomeffect voor een TL: het beschermt kwaliteit en onderwijs. Het doel is betere code en een sterkere team, geen gatekeeping. Reviews moeten snel, aardig en gericht zijn op wat ertoe doet.
Principes
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
Een concreet voorbeeld
In plaats van "Dit is fout, gebruik een map," schrijf je: "nit: een map hier zou de geneste lus vermijden, het waard? Niet blokkeren." Hetzelfde punt, maar het onderwijs, laat ruimte voor onenigheid en blokkeert de merge niet.
