GitHub çekme isteklerinde akran değerlendirmeleri nasıl yapılır?


12

Bitbucket'ten GitHub'a geçiyoruz ve mücadele ettiğimiz bir şey, Bitbucket'te çok düzgün çalışan akran kodu incelemeleridir:

  1. Yazar bir Çekme İsteği açtı (GitHub: aynı)
  2. Yazar meslektaşlarını hakem olarak ekledi (GitHub: ?? burada birden fazla atanan kişi ile mücadele ediyor )
  3. İnceleyen ya:
    1. PR'yi yeşil onay işaretiyle onayladı (GitHub: ??)
    2. Eklenen yorumlar (GitHub: aynı)
    3. Hafif görevler oluşturuldu (GitHub: - [ ]PR açıklamasında sözdizimi kullanılıyorsa bir çeşit benzer ; görevler için çalışmadığı için utanç verici)
  4. Bir bakışta gözden geçirilebilecek ve birleştirilecek ve daha fazla dikkat edilmesi gereken bir PR listesi var (GitHub: ??)

Mümkünse 3. taraf kod inceleme araçlarından kaçınmak istediğimizi ve bir çeşit geçici çözümle vanilya GitHub'da kalmak istediğimizi belirtmeliyim.


1
Zamanından önce geçiş yapıyormuş gibi görünüyorsun. Neden yeni bir şey ihtiyacınız olan tüm özelliklere sahip değilse, neden geçiş yapmalısınız?
dadı

Prq'inize bir yorum yazın ve bildirim almak istediğiniz her şeyi @ vurgulayın. İnceleyen, inceleme görüşlerini göstermek için etiketler ekleyebilir.
Wilbert

Yanıtlar:


6

Gördüğüm kadarıyla, bu adımların çoğu Github'da, resmi Github tarafından sağlanan herhangi bir süreç tarafından değil, sözleşmeyle yapılır.

İşverenim Github kullanıyor, çok sayıda küçük açık kaynaklı proje yürütüyorum ve diğer açık kaynaklı projelere ara sıra katkıda bulunuyorum.

Genelde böyle yapıldığını görüyorum:

Yazar meslektaşlarını hakem olarak ekliyor:

Bu projeden projeye değişir, ancak genel olarak atanan akran denetçileri projeye katkıda bulunanların tümüdür .

Açık kaynaklı projelerin kabaca bir hiyerarşisi varmış gibi görünebilir - belki de sözleşmeleri ancak “çekirdek” katkıda bulunanlara bir şey verdikten sonra birleşmek olacaktır.

Şu anda çalıştığım dükkanda, takımdaki yarım düzine geliştiriciden herhangi birinin onayını verdikten sonra birleşiyoruz.

Nadir durumlarda, ekipteki bir kişi, kodu birleştirilmeden önce akranı gözden geçirmesi gerektiğini düşündüğü başka bir geliştiriciyi özel olarak çağırmak için bir yorum kullanabilir, ancak aksi takdirde, oraya ilk gelen ve bunu yapan kişi yorum yapabilir ve yorum yapabilir.

Hakem onayı:

Onay genellikle "+1" veya "lgtm" (bana iyi görünür) diyen çekme isteği hakkında yorum yapılarak gösterilir .

Hafif görevler:

Onay kutularını da kullandım, ancak çoğu durumda, bir çekme isteğiyle ilgili her yorum şu şekilde çözülen örtük bir "görev" olarak kabul edilir:

  • satırın yorum yaptığı kodu değiştirme
  • başka bir yorumla cevap vermek

Nelerin onaylandığını ve nelerin hala gözden geçirilmesi gerektiğini bir bakışta görme:

Size Çekme İstekleri ekranından böyle bir görünüm veren Chrome için Bana İyi Bak uzantısını kullandım . Bununla birlikte, çekme istekleri listesi görünümü son Github değişikliklerinden kopmuş gibi görünüyor.

Sitemizi kullandığınızda şunları okuyup anladığınızı kabul etmiş olursunuz: Çerez Politikası ve Gizlilik Politikası.
Licensed under cc by-sa 3.0 with attribution required.