Kodgranskning är en av en TL:s högst påverkande aktiviteter: den skyddar kvalitet och undervisar. Målet är bättre kod och ett starkare team, inte grindvakt. Granskningar bör vara snabba, vänliga och fokuserade på det som spelar roll.
Principer
✓ 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
Ett konkret exempel
Istället för "Det här är fel, använd en map," skriver du: "nit: en map här skulle undvika den kapslade loopen, värt det? Inte blockerande." Samma poäng, men det undervisar, lämnar utrymme för oenighet och blockerar inte sammanslåningen.
