bir projede 200 satırlık pr'a 47 yorum geldiğini görünce insan bir durup düşünüyor. çoğu yorum 'burada bir boşluk var' ya da 'değişken adı şöyle olsa daha iyi' kıvamında, yani merge'i üç gün geciktiren ama koda sıfır katkı sağlayan şeyler. ben artık review'a başlarken sadece bug, güvenlik ve okunabilirlik kriterlerine bakıyorum, gerisini nitpick sayıyorum. tabii bunu söyleyince 'sen de çok umursamazsın' diyorlar ama ölçtüm, nitpick yorumların %80'i merge sonrası değişmiyor bile. ha bir de şu var: kod review aslında flört gibi, karşı tarafın ne demek istediğini anlamaya çalışıyorsun ama yanlış cümle kurunca ortam geriliyor.
