@noway · 22 gün önce
kod review'da neye bakıyorsunuz, bizim ekipte tartışma çıktı
Ekipte review süresi çok uzadı ve insanlar birbirine sinirlenmeye başladı. Oturup neye bakacağımızı yazdık, sonra tartışma bitti.
Sorunun kaynağı şuydu: herkes farklı şeye bakıyordu. Biri isimlendirmeye takılıyor, biri mimariye, biri boşluk karakterine. Aynı PR üç farklı yönden eleştiri alıyordu.
Şimdi üç kategoriye ayırdık ve yorumun başına etiket koyuyoruz.
Engelleyici. Birleştirilmemesi gereken şeyler. Güvenlik açığı, veri kaybı riski, yanlış mantık, eksik yetki kontrolü, testi olmayan kritik akış.
Öneri. Düzeltilirse iyi olur ama şart değil. Daha iyi bir yaklaşım, isimlendirme, tekrar eden kod.
Soru. Anlamadığım yer. Bazen cevap yeterli oluyor, bazen kodun açıklanması gerektiğini gösteriyor.
Biçimlendirme tartışmalarını tamamen kaldırdık. Otomatik biçimlendirici koyduk ve konu kapandı. İnsanların noktalı virgül tartışmasına harcadığı süre inanılmazdı.
Birkaç kural daha koyduk:
PR büyükse review kalitesi düşüyor. Dört yüz satırı geçen PR'ları bölmeyi istiyoruz.
Yazar kendi PR'ını önce kendi okuyor ve zor kısımlara açıklama yorumu bırakıyor. Bu tek alışkanlık review süresini yarıya indirdi.
İki günden fazla bekleyen PR için sorumluluk yazarda değil, review edende.
Tartışma üç mesajı geçerse yazışmayı bırakıp konuşuyoruz.
Sorunun kaynağı şuydu: herkes farklı şeye bakıyordu. Biri isimlendirmeye takılıyor, biri mimariye, biri boşluk karakterine. Aynı PR üç farklı yönden eleştiri alıyordu.
Şimdi üç kategoriye ayırdık ve yorumun başına etiket koyuyoruz.
Engelleyici. Birleştirilmemesi gereken şeyler. Güvenlik açığı, veri kaybı riski, yanlış mantık, eksik yetki kontrolü, testi olmayan kritik akış.
Öneri. Düzeltilirse iyi olur ama şart değil. Daha iyi bir yaklaşım, isimlendirme, tekrar eden kod.
Soru. Anlamadığım yer. Bazen cevap yeterli oluyor, bazen kodun açıklanması gerektiğini gösteriyor.
Biçimlendirme tartışmalarını tamamen kaldırdık. Otomatik biçimlendirici koyduk ve konu kapandı. İnsanların noktalı virgül tartışmasına harcadığı süre inanılmazdı.
Birkaç kural daha koyduk:
PR büyükse review kalitesi düşüyor. Dört yüz satırı geçen PR'ları bölmeyi istiyoruz.
Yazar kendi PR'ını önce kendi okuyor ve zor kısımlara açıklama yorumu bırakıyor. Bu tek alışkanlık review süresini yarıya indirdi.
İki günden fazla bekleyen PR için sorumluluk yazarda değil, review edende.
Tartışma üç mesajı geçerse yazışmayı bırakıp konuşuyoruz.
0 3 yorum