Mehmet
Admin
GitHub Copilot ile kod incelemesi yapmak için pull request'in inceleyici bölümünden Copilot'a review isteği gönderebilirsiniz. İşe yarar sonuç almak için önce değişikliğin amacını ve sınırını netleştirin. Bir öneriyi otomatik kabul etmek yerine, hangi kullanıcı davranışını bozduğunu gösterebilen bulgulara öncelik verin.
Bu açıklama gözden geçiren kişiye de yardımcı olur; yalnız yapay zekâ için hazırlanmış bir talimat değildir.
Örneğin “null kontrolü yok” bulgusunda yalnız kontrol eklemek yetmez. Null değer gerçekten bu katmana gelebiliyor mu, önceki doğrulama bunu engelliyor mu ve kullanıcıya dönülecek hata anlaşılır mı inceleyin. Sonra normal veri, eksik veri ve bozuk veriyle davranışı karşılaştırın.
Küçük bir sonuç notu şu biçimde tutulabilir:
İnceleme sonunda beklenen çıktı, uzun bir yorum listesi değil; doğrulanmış bulgular, bilinçli reddedilen öneriler ve çalıştığı gösterilmiş son davranıştır. İnsan incelemesini ve projenin gerekli testlerini bu yardımcı adımın yerine koymayın.
İnceleme istemeden önce değişikliği daraltın
Tek pull request içine hem dosya yeniden adlandırma hem paket yükseltme hem yeni özellik koymak incelemeyi zorlaştırır. Mümkünse bağımsız işleri ayırın. Açıklamaya “önceden ne oluyordu, şimdi ne olacak?” sorusunun cevabını ekleyin. Örneğin:
Kod:
Problem: Boş sepetle ödeme sayfası açılıyor.
Değişiklik: Sepet boşsa ödeme isteği başlatılmadan uyarı gösteriliyor.
Kontrol: Boş sepet, tek ürün, miktarı sıfırlanan ürün.GitHub üzerinden review isteyin
- İncelenecek pull request'i açın ve doğru hedef dalı seçtiğinizi doğrulayın.
- Sağdaki Reviewers alanında Copilot yanındaki Request seçeneğini kullanın. Hesabınızda bu seçenek yoksa önce erişim ve kuruluş ayarlarını kontrol edin.
- Gelen yorumları dosya ve satır bağlamıyla okuyun. “High” gibi bir önem etiketi gördüğünüzde de etkisini kod akışıyla doğrulayın.
- Uygulamayı düşündüğünüz öneriyi küçük bir değişiklik olarak ele alın; ilgisiz biçimlendirmeleri aynı düzeltmeye eklemeyin.
Gerçek hata ile tercihi ayıran üç soru
Bir yorum için şu soruları yanıtlayın: Hangi girdi bu hatayı tetikler? Çalışan kodda sonuç ne olur? Önerilen değişiklikten sonra aynı girdi ne üretir? Bu zincir kurulamıyorsa yorum bir isimlendirme tercihi veya henüz doğrulanmamış ihtimal olabilir.Örneğin “null kontrolü yok” bulgusunda yalnız kontrol eklemek yetmez. Null değer gerçekten bu katmana gelebiliyor mu, önceki doğrulama bunu engelliyor mu ve kullanıcıya dönülecek hata anlaşılır mı inceleyin. Sonra normal veri, eksik veri ve bozuk veriyle davranışı karşılaştırın.
Yeni commit'ten sonra eski incelemeye güvenmeyin
Kod değiştiğinde önceki yorumun hâlâ geçerli olduğunu kontrol edin. Copilot yeni commit'leri otomatik yeniden incelemiyorsa Reviewers alanından tekrar inceleme isteyin; otomatik inceleme davranışı depo ayarlarına bağlı olabilir. “Resolve conversation” düğmesi, hatanın düzeldiğine dair teknik kanıt değildir.Küçük bir sonuç notu şu biçimde tutulabilir:
Kod:
Bulgu: Boş sepette ödeme isteği oluşuyor.
Karar: Kabul edildi.
Kanıt: Ağ isteği önce oluşuyor, düzeltmeden sonra oluşmuyor.
Ek kontrol: Tek ürünle ödeme akışı devam ediyor.