Bu belgede, bir değişikliğin nasıl inceleneceğiyle ilgili talimatlar verilmektedir. Geçerli bir değişiklik için aşağıdaki ölçütleri göz önünde bulundurun:
- Bu değişiklik, projenin belirtilen amacı kapsamında mı?
- Bu değişiklik, projenin mevcut mimarisi içinde geçerli mi?
- Bu değişiklik, gelecekte sorunlara neden olabilecek tasarım hatalarına yol açıyor mu?
- Bu değişiklik, bu proje için belirlenen en iyi uygulamalara uygun mu?
- Bu değişiklik, açıklanan işlevi gerçekleştirmek için iyi bir yöntem mi?
- Bu değişiklik, güvenlik veya kararsızlık risklerine yol açıyor mu ya da mevcut uygulamalarda herhangi bir bozulmaya neden oluyor mu?
Değişiklik inceleme hakkında daha fazla bilgi için Gerrit belgelerinin Değişiklik inceleme bölümüne bakın.
Bir değişikliğin inceleyeni olarak atandıysanız değişikliğin bağlantısını içeren bir e-posta alırsınız. Değişikliği incelemek ve onaylamak için aşağıdaki adımları uygulayın:
Değişikliğin e-postadaki bağlantısını tıklayın. Gerrit'in Değişiklik Bilgileri ekranı gösterilir. Bu ekranda değişikliğin açıklaması, değişikliğe katkıda bulunan ve tüm inceleme uzmanları belirtilir. Ayrıca, değişiklikle ilişkili dosyaların listesi de gösterilir.
Değişikliğin amacını anlamak için değişiklik açıklamasını okuyun.
Dosyadaki değişiklikleri incelemek için dosyayı tıklayın. Dosya, solda mevcut dosya, sağda ise önerilen değişikliklerin bulunduğu dosya olacak şekilde iki bölmeli bir karşılaştırma ekranında açılır. Dosyayı incelemek için:
Fareyle, yorum veya soru eklemek istediğiniz kod bölümünü vurgulayın (tıklayıp sürükleyin). "Yorum yapmak için c tuşuna basın" mesajı gösterilir.
c tuşuna basın. Taslak yorum kutusu gösterilir.
Yorumunuzu veya sorunuzu girip KAYDET'i tıklayın.
Değişiklik Bilgileri ekranına dönmek için değişikliğin benzersiz numarasını (ekranın sol üst tarafı) tıklayın.
Değişiklikteki her dosyayı incelemeye devam edin.
Değişiklikteki tüm dosyaları incelemeyi bitirdiğinizde Değişiklik Bilgileri ekranına dönüp YANITLA'yı tıklayın. Gereksinim oylarını gönderme ekranı görünür.
Değişiklik için aşağıdaki puanlardan birini seçin:
- +1 Bana göre iyi görünüyor ancak başka birinin de onaylaması gerekiyor.
0 Puan yok
-1 Bu gönderinin olduğu gibi gönderilmemesini tercih ederim, düzeltilmesi gereken şeyler var.
-2 Bu değişiklik gönderilmemeli
Yorumlarınızı ve puanınızı değişikliği yapan geliştiriciye göndermek için YANITLA'yı tıklayın.
Düzeltilmesi gereken sorunlar varsa geliştirici bunları düzeltir ve düzeltmeleri incelemenizi ister. Aksi takdirde, değişikliği inceleme işlemini tamamlamış olursunuz.