Hvordan udfører du effektive og respektfulde koderevisioner?

Question

Accepted Answer

Kodereview er en af en teamlederes højst gearværksaktiviteter: det beskytter kvalitet *og* lærer teamet. Målet er **bedre kode og et stærkere team**, ikke gatekeeping. Anmeldelser skal være hurtige, venlige og fokuseret på det, der betyder noget.

## Principper

```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 "Det er forkert, brug et kort," skriver du: "nit: et kort her ville undgå den indlejrede løkke, værd det? Blokerer ikke.". Samme pointe, men det lærer, efterlader plads til uenighed og blokerer ikke sammenlægningen.

## Pas på magtdynamikken

Som teamleder har dine kommentarer ekstra vægt. En casual "Jeg ville gøre det anderledes" kan føles som en ordre. Vær eksplicit om hvad der er et reelt problem versus præference, og lad folk gøre indsigelse.

## Hvorfor det betyder noget

Anmeldelser fastlægger teamets kvalitetsmåler og følelsesmæssige tone.

Strenge anmeldelser gør mennesker defensive og langsomme til at levere.

Gode anmeldelser fanger fejl, spreder viden og gør ingeniører bedre, sammensat over tid.