Bu dokümanda, bir değişikliğin nasıl inceleneceği ile ilgili talimatlar yer almaktadır. Geçerli bir değişiklik için aşağıdaki kriterleri göz önünde bulundurun:
- Bu değişiklik projenin belirtilen amacına uygun mu?
- Bu değişiklik projenin mevcut mimarisi için geçerli mi?
- Bu değişiklik, ileride sorunlara yol açabilecek tasarım hatalarına yol açıyor mu?
- Bu değişiklik bu proje için belirlenen en iyi uygulamaları izliyor mu?
- Bu değişiklik, açıklanan işlevi gerçekleştirmek için iyi bir yol mu?
- Bu değişiklik herhangi bir güvenlik veya kararsızlık riskine yol açar mı ya da mevcut uygulamaları bozar mı?
Bir değişikliği inceleme hakkında ek bilgi için Gerrit belgesinin Bir değişikliği inceleme bölümüne bakın.
Bir değişiklik için incelemeci 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 incelemek için aşağıdaki adımları izleyin:
Değişiklik için e-posta bağlantısını tıklayın. Gerrit'in Bilgileri Değiştir ekranı görüntülenir. Bu ekranda, değişiklikle ilgili bir açıklama, değişikliğe katkıda bulunan kişi ve tüm inceleyiciler belirtilir ve değişiklikle ilişkili dosyaların bir listesi sunulur.
Değişimin amacını anlamak için değişiklik açıklamasını okuyun.
Bir dosyadaki değişiklikleri incelemek için dosyayı tıklayın. Dosya, sol tarafta mevcut dosya, sağ tarafta 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:
Kodun yorum yaptığınız veya sorunuz olan bölümünü vurgulamak (tıklayıp sürükleyin) için farenizi kullanın. "Yorum yapmak için c tuşuna basın" mesajı görünür.
c tuşuna basın. Bir yorum taslağı kutusu görüntülenir.
Yorumunuzu veya sorunuzu girip KAYDET'i tıklayın.
Bilgileri Değiştir ekranına dönmek için değişikliğin benzersiz numarasını (ekranın sol üst tarafında) tıklayın.
Değişiklikteki her bir dosyayı incelemeye devam edin.
Değişiklikteki tüm dosyaları incelemeyi bitirdiğinizde Bilgileri Değiştir ekranına dönün ve YANITLA'yı tıklayın. Gereksinim oylarını gönder ekranı görünür.
Değişiklik için aşağıdaki puanlardan birini seçin:
- +1 bana iyi görünüyor, ancak başka birinin de onaylaması gerekiyor.
0 Puan yok
-1 Bunun olduğu gibi gönderilmemesini tercih ederim, düzeltilmesi gereken şeyler var.
-2 Bu değişiklik gönderilmemelidir
Yorumlarınızı ve puanınızı, değişikliğin geliştiricisine 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 incelemeyi tamamlamış olursunuz.