Bir Pull Requesti Kıdemli Bir Geliştirici Gibi İncelemek

Mehmet Oya
Papara Tech
Published in
3 min readJul 31, 2024

Pull request, kod tabanına bir dizi değişiklik yapılacağı zaman bu değişikliklerin değerlendirilmesi, tartışılması ve son onayın verilmesi amacıyla kullanılır. Bir yazılım geliştirme ekibi içinde, pull request incelemeleri, kod kalitesini artırmak, hataları tespit etmek ve işbirliğini desteklemek için kritik bir süreçtir. Kıdemli geliştiriciler, bu incelemelerde öncü bir rol oynar, çünkü deneyimleri ve bilgileri, ekibin kod tabanını geliştirmesine katkı sağlar.

Pull Request Review

1.İncelemeye Başlarken

a. Açıklamaları Okuyun

Pull request açıklamalarını ve değişiklikleri anlatan yorumları dikkatlice okuyun. Bu, değişikliklerin niçin yapıldığını ve nasıl test edildiğini anlamanıza yardımcı olur

b. İşlevi Anlayın

Değişikliklerin ne yaptığını anlamak için kodu gözden geçirmeden önce genel işlevi anlamaya çalışın. Bu, kodun neyin üzerinde çalıştığını ve hangi amaçla tasarlandığını anlamanıza yardımcı olur.

2. Kod Kalitesini İnceleyin

a. Temizlik ve Okunabilirlik

Kodun okunabilir olması ve temiz bir biçimde yazılmış olması, sürdürülebilirlik açısından önemlidir. Değişken isimleri, işlevlerin adları ve kod yapısı açık ve anlaşılır olmalıdır.

b. Mantıklı Bölümleme

Kod parçalarının mantıklı bir şekilde bölünmesi, işlevlerin ve sınıfların belirli bir amacı yerine getirmesi gerektiği anlamına gelir. Aşırı uzun işlevler veya sınıflar karmaşıklığa yol açabilir.

c. Tekrar Edilen Kodları Azaltın

Tekrar eden kod parçalarını tespit edin ve bunları uygun şekilde soyutlayarak veya yeniden kullanarak kodu temizleyin.

3. Hata ve Güvenlik Kontrolü

a. Hata ve Uyum İhlallerini Kontrol Edin

Kodun hatalı çalışma durumlarına yol açabilecek potansiyel sorunları olup olmadığını kontrol edin. Null referanslar, hatalı tür dönüşümleri gibi hataları tespit edin.

b. Güvenlik Kontrolü

Kodun güvenli olup olmadığını kontrol edin. Veri doğrulama, kimlik doğrulama ve yetkilendirme gibi güvenlik önlemlerine uyulduğundan emin olun.

4. Performans ve İyileştirme

a. Performans Değerlendirmesi

Kodun performansını değerlendirin. Büyük veri işlemleri veya döngüler gibi performans etkileyen bölümleri belirleyin.

b. İyileştirme Önerileri

Kodun performansını veya verimliliğini artıracak önerilerde bulunun. Örneğin, veritabanı sorgularını optimize etme veya döngüleri daha etkili bir şekilde kullanma.

5. Testler ve Dökümantasyon

a. Test Kapasitesi

Değişikliklerin test edildiğinden emin olun. Hem yeni testler eklenmesi hem de mevcut testlerin geçmesi gereklidir.

b. Dökümantasyon

İlgili dökümantasyonun güncellenip güncellenmediğini kontrol edin. API belgeleri, README dosyası ve gerektiğinde kod içi açıklamalar gözden geçirilmelidir.

6. İnceleme Geri Bildirimi

a. Olumlu ve İyileştirici Geri Bildirim

İnceleme sırasında olumlu yönleri övmek yanında, kodun daha iyi hale getirilmesi için yapıcı önerilerde bulunun.

b. Açıklamalarla Destekleyin

İnceleme yorumlarınızı açıklayıcı bir şekilde sunun. Eğer bir değişiklik yapılması gerekiyorsa, nedenini ve alternatif çözüm önerilerini açıklayın.

7. İşbirliği ve Ekip Ruhu

a. Açık İletişim

İnceleme sürecinde açık iletişim sürdürün. Geliştiricilere sorular sormaktan veya çekincelerinizi paylaşmaktan çekinmeyin.

b. İşbirliğine Açık Olun

İnceleme süreci, birlikte çalışmanın bir parçasıdır. İncelemeyi yapan ve incelemeye tabi olan geliştirici arasında işbirliğini teşvik edin

Kaynaklar

GitHub Pull Request Review Guide

GitHub Pull Request Review sayfası, GitHub üzerinde pull request incelemesi yapma sürecini adım adım açıklar. GitHub’un özelliklerini kullanarak nasıl etkili bir inceleme yapılacağına dair bilgi verir.

Bitbucket Pull Request Code Review

Bitbucket Code Review sayfası, Bitbucket kullanıcıları için pull request incelemesi hakkında bilgi sunar. Bitbucket’in sağladığı özelliklerle nasıl daha iyi bir inceleme süreci yürütülebileceği konusunda rehberlik eder.

GitLab Merge Request Workflow

GitLab Merge Request Workflow dökümantasyonu, GitLab kullanıcılarına pull request ve merge request kavramını anlatır. Merge request sürecini nasıl başlatacağınızı ve incelemeleri nasıl yöneteceğinizi açıklar.

Understanding Code Review in Open Source

Open Source Guides platformu, açık kaynak projelerinde kod incelemesini anlamak ve yönetmek için rehberlik eder. Pull request incelemesi yaparken dikkate almanız gereken açık kaynak topluluğunun özel dinamiklerini ele alır.

The Art of Pull Requests

The Art of Pull Requests makalesi, etkili bir pull request incelemesi yapmak için izlenecek yaklaşımları açıklar. İyi bir inceleme sürecinin nasıl yönetileceğini ve katkıda bulunanların nasıl destekleneceğini anlatır.

Bu kaynaklar, pull request incelemesi hakkında daha fazla bilgi edinmek ve en iyi uygulamaları öğrenmek için faydalı olacaktır. Her bir kaynak, farklı platformlar ve süreçler hakkında bilgi sunarak daha geniş bir perspektif sağlar.

--

--

Mehmet Oya
Papara Tech

I am an Istanbul based Software Developer with 10+ years of experience. I am passionate about building software that improves the lives of those around me.