Hur genomför du effektiva och respektfulla kodgranskningar?

Question

Accepted Answer

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

```text
✓ 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.

## Observera maktdynamiken

Som TL har dina kommentarer extra vikt. En casual "Jag skulle göra det annorlunda" kan kännas som ett order. Var explicit om vad som är en verklig angelägenhet kontra en preferens, och låt människor göra invändningar.

## Varför det spelar roll

Granskningar sätter teamets kvalitetsmässiga standard och dess känslomässiga ton.

Stranga granskningar gör människor defensiva och långsamma att leverera.

Bra granskningar fångar buggar, sprider kunskap och gör ingenjörer bättre, vilket förstärks över tid.