Takımınızda kod inceleme süreci nasıl organize…

Versiyon kontrol sistemi Git ve depo yönetim platformları (örneğin, Bitbucket veya GitLab) kullanarak ortak geliştirme yaptık. Kod inceleme süreci şu şekilde organize edildi:

Dal oluşturma: Geliştirici, yeni özellik veya hata düzeltmesi için ayrı bir dal oluşturur (git checkout -b feature/my-new-feature).
Geliştirme ve commitler: Dal üzerinde değişiklikler yapılır ve commitlerle kaydedilir (git add ., git commit -m "İlk özellik uygulaması").
Uzak depoya push: Dal uzak depoya gönderilir (git push origin feature/my-new-feature).
Pull/Merge Request oluşturma: Geliştirici, hedef dalı belirterek (örneğin, develop veya main) Pull Request (Bitbucket'ta) veya Merge Request (GitLab'da) oluşturur. Açıklamada değişikliklerin kısa özeti, görev takip araçlarındaki (Jira, Trello vb.) bağlantılar ve ekli dosyalar (ekran görüntüleri, videolar, diyagramlar) yer alır.
İnceleme yapanların atanması: Takımdan bir veya iki geliştirici inceleme yapmak üzere atanır.
İnceleme süreci: İnceleyenler, Pull/Merge Request'taki değişiklikleri gözden geçirir. Yorumlar bırakabilir, geliştirmeler önerebilir, sorular sorabilirler.
```
// İnceleme sırasında örnek yorum
function fetchData() {
  // Belki burada veri isteğinde hata yönetimi eklenmeli
  return fetch('/api/data');
}
```
İnceleme sonuçlarına göre değişiklikler: Geliştirici, inceleyenlerin yorumlarına dayanarak gerekli değişiklikleri yapar ve yeni commitler yapar.
Pull/Merge Request güncellemesi: Değişiklikler, uzak depoya push yapıldıktan sonra otomatik olarak Pull/Merge Request'ta gösterilir.
Yeniden inceleme: İnceleyenler, değişiklikleri tekrar gözden geçirir ve onay verir.
Dal birleştirme: Onay alındıktan sonra, dal hedef dala (genellikle platformun CI/CD pipeline'ından geçtikten sonra otomatik olarak) birleştirilir.

Ayrıca, CI/CD pipeline'ları kullanarak birim testleri, statik kod analizi (ESLint, Prettier) ve projenin derlenmesini otomatik olarak çalıştırdık, bu da hataları erken aşamada tespit etmeye yardımcı oldu.

Başarılı inceleme kriterleri şunları içeriyordu:

Görev gereksinimlerine uygunluk.
Takımın kod stiline uyum.
Kritik mantık için birim testlerinin bulunması.
Açık hatalar ve "geçici çözümler" olmaması.
Kodun diğer takım üyeleri tarafından anlaşılır olması.

Takımınızda kod inceleme süreci nasıl organize edildi?