Koodin tarkistaminen on yksi tekniikan johtajalle korkeimmasta vipuvaikutuksesta: se suojaa laatua ja opettaa. Tavoitteena on parempi koodi ja vahvempi tiimi, ei porttipidättö. Tarkistuksien tulisi olla nopeita, ystävällisiä ja kohdistettuja siihen, mikä on tärkeää.
Periaatteet
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
Konkreettinen esimerkki
Senhetkisen "Tämä on väärin, käytä mappia", kirjoita: "nit: mapin käyttö täällä välttäisi sisäkkäisen silmukan, kannattaa? En estä sulautumista." Sama pointti, mutta se opettaa, jättää tilaa olla eri mieltä eikä estä sulautumista.