Hvordan gjennomfører du effektive og respektfulle kodeanmeldelser?

Question

Accepted Answer

Kodeanmeldelse er en av TLs høyeste-leverage-aktiviteter: den beskytter kvalitet *og* lærer. Målet er **bedre kode og et sterkere team**, ikke gatekeeping. Anmeldelser bør være raske, vennlige og fokusert på det som betyr noe.

## Prinsipper

```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
```

## Et konkret eksempel

I stedet for "Dette er feil, bruk et kart," skriver du: "nit: et kart her ville unngå den nestede løkken, verdt det? Ikke blokkerer." Samme poeng, men det lærer, etterlater rom for uenighet, og blokkerer ikke sammenslåingen.

## Se opp for maktdynamikken

Som TL har kommentarene dine ekstra vekt. En uformell "Jeg ville gjort det annerledes" kan føles som en ordre. Vær eksplisitt om hva som er en reell bekymring versus en preferanse, og la folk gjøre motstand.

## Hvorfor det betyr noe

Anmeldelser setter teamets kvalitetsstandarder og dets emosjonelle tone.

Harde anmeldelser gjør folk defensive og trege til å levere.

Gode anmeldelser fanger feil, sprer kunnskap og gjør ingeniører bedre, og sammensetter seg over tid.