Bu commit, PR veya MR’da gerçekleşen "değişiklik" hakkında, o değişikliğin uygun olup olmadığına dair yorum bırakmak kolay; ancak değiştirilmesi gerekiyordu ama değiştirilmemiş kısımları fark edip yorum bırakmak, kod incelemesini oldukça uzun süredir yapıyor olmama rağmen hâlâ zor geliyor. Bana çok tanıdık bir kod tabanı değilse bu daha da zor oluyor.
Aklıma birkaç yöntem geliyor ama daha fazla mekanizma olsa iyi olur diye düşünüyorum. Siz bunu nasıl yapıyorsunuz, merak ediyorum.
- Uygularken: Sonradan birlikte değiştirilmesi gereken kısımlar, test kodunun bunları birlikte kapsayabileceği şekilde uygulanır.
- PR yazarken: PR şablonuna, incelemeyi yapan kişinin değerlendirebileceği uygun sorular eklenir. (örn. Bu PR ile benzer amaca sahip önceki bir PR var mı? O PR ile karşılaştırdığınızda bu PR’da acaba atlanmış bir kısım var mı?)
- O kod tabanında 'yapılması gerekiyordu ama atlanmış' bir durum her ortaya çıktığında bunu kaydedip, her seferinde PR şablonundaki ilgili örnekleri güncelleriz.
- PR incelerken: İnceleyenin, yukarıdaki sorulara benzer şekilde eksik kalan kısımları aramasını sağlayacak maddeleri inceleme rehberine ekleriz.
Henüz yorum yok.