Bu dokümanda, bir değişikliğin nasıl inceleneceğiyle ilgili talimatlar yer almaktadır. Lütfen şu ölçütleri dikkate almanız gerekir:
- Bu değişiklik projenin belirtilen amacına uygun mu?
- Bu değişiklik, projenin mevcut mimarisinde geçerli mi?
- Bu değişiklik, gelecekte soruna yol açabilecek tasarım kusurları içeriyor mu?
- Bu değişiklik, 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 kararlılık riskleri oluşturuyor mu ya da mevcut uygulamaların çalışmasını engelliyor mu?
Bir değişikliği inceleme hakkında daha fazla bilgi için Gerrit dokümanlarının Değişikliği inceleme bölümüne bakın.
Bir değişikliğin incelemecisi olarak atanırsanız değişikliğin bağlantısını içeren bir e-posta alırsınız. Değişikliği incelemek için aşağıdaki adımları uygulayın:
Değişiklik için e-posta bağlantısını tıklayın. Gerrit'in Değişim Bilgisi görüntülenir. Bu ekranda, değişiklikle ilgili bir açıklama yer alır. değişimine katkıda bulunan kişiyi ve tüm inceleyicileri belirler ve değişikliklerin bir listesini sunar dosyalardan kaçının.
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 dosyanın, sağda ise önerilen değişikliklerin yer aldığı iki bölmeli bir karşılaştırma ekranında açılır. Dosyayı incelemek için:
Farenizi kullanarak kodun hangi bölümünü vurgulamak istediğinizi (tıklayıp sürükleyin) bir yorumunuz veya sorunuz varsa. "Yorum yapmak için c tuşuna basın" mesajı gösteriliyor görünür.
c tuşuna basın. Taslak yorum kutusu görünür.
Yorumunuzu veya sorunuzu girip KAYDET'i tıklayın.
Dönmek için değişikliğin benzersiz numarasını (ekranın sol üst tarafında) tıklayın ve Bilgileri Değiştir ekranına gidin.
Değişiklikteki her bir dosyayı incelemeye devam edin.
Değişiklikteki tüm dosyaları incelemeyi bitirdiğinizde Bilgileri Değiştir ekranına gidip YANITLA'yı tıklayın. Gönderme koşulu oyları 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.