Uzun süredir takım liderliği yapıyorum, acele etmeden yazayım çünkü bu konuda en çok yapılan hata aceleye getirmek.
Kod incelemesi denetim değil, eğitim aracıdır. Bu cümleyi kabul etmeyen ekiplerde süreç her zaman aynı yere gidiyor: kıdemli kişi hata avlar, genç kişi savunmaya geçer, incelemeler yavaşlar, sonunda kimse bakmadan onaylar.
İşe yarayan birkaç kural biriktirdim, paylaşayım.
Küçük parçalar halinde gönderin. Beş yüz satırlık bir değişikliği kimse gerçekten okuyamaz, herkes "iyi görünüyor" yazar. İki yüz satırın altında kalan değişikliklerde bulunan sorun sayısı belirgin şekilde artıyor.
Yorumu kişiye değil koda yazın. "Bunu neden böyle yaptın" yerine "burada şu durumda ne olur" diye sorun. Aynı bilgi, bambaşka bir konuşma.
Tarz tartışmasını otomatikleştirin. Girinti, tırnak, isimlendirme gibi konular araçla halledilmeli. İnsan zamanı bunlara harcanırsa, gerçek sorunlar gözden kaçıyor.