Kod İncelemeleri ve Onaylar

Kod incelemeleri, modern yazılım geliştirmenin temel taşlarından biridir. Geliştirmenin ilk aşamasında gözden kaçan hataları bulup düzeltmek için kaynak kodun ekip arkadaşları tarafından sistematik biçimde incelenmesini sağlar.

Temel hedefler şunlardır:

• Kod kalitesini iyileştirme: Hataları, güvenlik açıklarını ve tasarım kusurlarını erkenden yakalamak.
• Bilgi paylaşımı: Kod tabanının anlaşılma düzeyini ekip geneline yaymak.
• Mentorluk: Deneyimli geliştiricilerin daha az deneyimli geliştiricilere yol göstermesi ve herkesin farklı bakış açılarından öğrenmesi.

GitHub'da kod inceleme süreci genellikle şu adımları izler:

1. Bir yazar, değişikliklerini içeren bir Çekme İsteği (PR) oluşturur.
2. Yazar inceleyiciler atar veya inceleme ister.
3. İnceleyiciler kodu inceler, yorumlar ve öneriler bırakır.
4. Yazar geri bildirimleri ele alır ve PR dalına yeni işlemler gönderir.
5. İnceleyiciler memnun kaldığında değişiklikleri onaylar.
6. Son olarak PR, ana dalla birleştirilir.

İyi bir kod incelemesi yalnızca hataları bulmakla ilgili değildir; projenin genel sağlığını iyileştirmeyi de amaçlar. İnceleme yaparken şunları göz önünde bulundurun:

• Doğruluk: Kod yapması gerekeni yapıyor mu? Sınır durumları var mı?
• Okunabilirlik: Anlaşılması kolay mı? Değişken adları açık mı?
• Bakımı kolay olma: Başkaları bu kodu daha sonra kolayca değiştirebilir veya genişletebilir mi?
• Performans: Belirgin verimsizlikler var mı?
• Güvenlik: Herhangi bir güvenlik açığı oluşturuyor mu?

Bir inceleyici olarak geri bildiriminiz her zaman yapıcı ve saygılı olmalıdır. Kodu, onu yazan kişiyi değil, incelediğinizi unutmayın.

Geri bildirim vermek için ipuçları:

• Belirli olun: Kodun tam satırlarını gösterin.
• 'Neden'i açıklayın: Yalnızca 'bunu değiştirin' demeyin; neden değiştirilmesi gerektiğini açıklayın.
• Çözümler önerin: Alternatif yaklaşımlar veya kod parçacıkları sunun.
• İyi niyetli olun: Kibar bir dil kullanın ve karşı tarafın iyi niyetli olduğunu varsayın.

GitHub, inceleme sürecini kolaylaştırmak için güçlü araçlar sağlar:

• Satır yorumları: Doğrudan yorum eklemek için 'Değiştirilen dosyalar' sekmesindeki bir satıra tıklayın.
• Öneriler: Yazarın tek tıklamayla uygulayabileceği belirli kod değişiklikleri önerebilirsiniz.
• Özet inceleme: Son aşamada 'Yorum', 'Onayla' veya 'Değişiklik iste' durumlarından biriyle özet inceleme gönderebilirsiniz.

Öneriler özellikle küçük ve anlaşılır iyileştirmeler için kullanışlıdır:

// Original Code
- const count = 0;
+ const initialCount = 0; // Better name

Bir Çekme İsteğinin yazarıysanız geri bildirimlere yanıt vermeniz çok önemlidir. Bu, sürece dahil olduğunuzu ve kodu iyileştirmeye kararlı olduğunuzu gösterir.

Yorumları ele alırken:

• Kabul edin: Yalnızca 'İyi nokta!' veya 'Tamamlandı' demek için bile olsa her yoruma yanıt verin.
• Değişiklik yapın: PR dalınıza yeni işlemler gönderin. GitHub PR'ı otomatik olarak günceller.
• Konuşmaları çözümleyin: Bir yorum ele alındığında GitHub'da 'Çözüldü' olarak işaretleyin.
• Sorular sorun: Bir öneriyi anlamadıysanız açıklama isteyin.

'Onayla' durumu, bir inceleyicinin Çekme İsteğindeki değişikliklerden memnun olduğunu açıkça gösterir. Birçok depo, PR'ın birleştirilebilmesi için en az bir (veya daha fazla) onay gerektirecek şekilde yapılandırılmıştır.

Onay, inceleyicinin kodun şu koşulları karşıladığına inandığını gösterir:

• Gereksinimleri karşılar.
• İyi yazılmıştır ve bakımı kolaydır.
• Tüm önemli endişeler ele alınmıştır.

Bu, bütünleştirme için yeşil ışıktır!

Bazen bir Çekme İsteğinin birleştirilebilmesi için daha fazla çalışma gerekir. Böyle durumlarda inceleyici, inceleme durumu olarak 'Değişiklik iste' seçeneğini belirleyebilir.

Bu durum şunları açıkça belirtir:

• Çözülmesi gereken ve birleştirmeyi engelleyen sorunlar vardır.
• Bu değişiklikler yapılana ve inceleyici yeni bir onay verene kadar PR birleştirilemez.

Sorunlar önemli olduğunda ve kodun kabul edilebilir olmasını engellediğinde bu seçeneği kullanın.

Kod incelemelerinden en iyi şekilde yararlanmak için hem yazarlar hem de inceleyiciler bazı en iyi uygulamaları izlemelidir:

• PR'ları küçük tutun: Küçük PR'ların incelenmesi daha kolay ve hızlıdır.
• Açık açıklamalar yazın: Yazarlar ayrıntılı PR açıklamaları ve bağlam sağlamalıdır.
• Hızlı davranın: İnceleyiciler hızlıca incelemeye, yazarlar da zamanında yanıt vermeye çalışmalıdır.
• Yapabildiklerinizi otomatikleştirin: Basit sorunları incelemeden önce yakalamak için kod denetleyicileri ve otomatik testler kullanın.
• İncelemelerden öğrenin: Her incelemeyi öğrenmek ve gelişmek için bir fırsat olarak görün.

Öğrendiklerimize göre aşağıdakilerden hangileri kod incelemelerine katılım için iyi uygulamalar olarak kabul edilir?

Kod incelemeleri ve GitHub'ın onay özelliklerini keşfettiniz! İncelemelerin kod kalitesi ve bilgi paylaşımı için neden önemli olduğunu, tipik inceleme akışını ve iyi inceleme uygulamalarının temel unsurlarını ele aldık.

Her zaman yapıcı geri bildirim vermeyi, öneriler gibi GitHub araçlarından yararlanmayı ve hem inceleme hem de geri bildirimlere yanıt verme süreçlerinde en iyi uygulamaları izlemeyi unutmayın. Onaylar ve 'Değişiklik iste' durumu, PR'larınızı etkili biçimde yönetmek için kritik sinyallerdir.

Değerli bir iş ortağı olmak için bu becerileri geliştirmeye devam edin!