GitHub'da, gözden geçiren ve vekil arasındaki fark nedir?

GitHub blogunda duyurulan, 7 Aralık 2016'da eklenen bir özellik, bir Çekme İsteğine yorum ekleme seçeneğini tanıttı

Artık ortak çalışanlardan açıkça bir inceleme isteyebilirsiniz, böylece çekme isteğinizi kimin incelemek istediğinizi belirtmeniz kolaylaşır.

Ayrıca, çekme isteği sayfası kenar çubuğundan incelemeyi beklediğiniz kişilerin bir listesini ve bunları daha önce bırakmış olanların incelemelerinin durumunu da görebilirsiniz.

Bununla birlikte, bir halkla ilişkiler için bir gözden geçirenin açık bir şekilde ayarlanması, insanlar atanarak zaten yapılmıştır ( atananlar seçeneği).

Her iki seçeneğin de mevcut olmasıyla, her ikisi de aynı son hedefi paylaştıkları için her seçeneğin rolü nedir?

DÜZENLE:

Birkaç OSS koruyucusu ile görüştükten sonra, gözden geçirenler kelimenin ne olması gerektiği olarak tanımlanır: gözden geçirmek (birinin kodu) ve "vekil" aşağıda daha gevşek bir tanımlamaya sahiptir.

"İnceleyen" için : kodu gözden geçirmek istediğiniz biri. Bu alandan sorumlu olan veya taahhüdü birleştirmekten sorumlu olan kişi olmak zorunda değildir. GitHub'ın önerdiği gibi, daha önce bu kod parçası üzerinde çalışan biri olabilir.

"Atanan" için : projenin ekibine / bakıcısına ne anlama geldiğini ve kesin bir tanım olmadığını. Halkla İlişkiler açıcısı veya bu alandan sorumlu biri (inceleme tamamlandıktan sonra veya sadece kapatıldıktan sonra Halkla İlişkileri kabul edecek) olabilir. Proje yöneticileri için projesine en uygun olanı neyin açık bıraktığını tanımlamak GitHub'a bağlı değildir.

Önceki cevap:

Tamam, kendi sorumu cevaplayacağım.

Yazma erişimine sahip kullanıcıların PR'ları için : Atanan, PR'yi açan aynı kişi olacaktır ve gözden geçiren, eski atanan fonksiyonun (kod gözden geçirme) yerini alacaktı ve bu da atanan kişinin seçtiği biri olacaktı.

Yazma erişimi olmayan kullanıcıların Halkla İlişkileri için (katkıda bulunanlar dışında): Yazma erişimi olan biri, Halkla İlişkileri gözden geçirmek için kendisini (veya başka bir yazma imtiyaz üyesini) tayin eder. Atanan boş.

Dış katılımcılar tarafından tamamlanmamış PR için : yazma erişim üyesi tamamlanmamış işi alıp ona tayin ederdi. Görevi bitirmek, Atanmak olmaktan sorumlu olacaktır . PR'ların ana nedeni değişiklikleri gözden geçirmek olduğundan, değişiklikleri incelemek için başka kişileri seçerdi.

GitHub'da gözden geçiren kişi, çekme isteğini inceleyen kişidir. Bir proje sahibi, bakıcılardan herhangi birinden inceleme talep edebilir, Çekme talebinin yalnızca yazma erişimi olan bakıcılardan biri tarafından gözden geçirilmesi durumunda birleştirilebilmesi için bir seçenek bile ayarlayabilirler.

Resmi github belgelerine göre , Assignee belirli konular ve çekme talepleri üzerinde çalışan bir kişidir. Bazen gözden geçiren olarak karıştırılır. Aslında, bir istek aldığımızda sorunu düzeltmek için birini atayabilmemiz için çekme isteği yerine sorunlarla kullanılmak üzere tasarlanmıştır. Çekme talebinde, bir görevli, yorumlar aldıktan ve diğer koruyuculardan gelen talepleri değiştirdikten sonra bu çekme isteğini birleştirmekten sorumlu olan kişiyi ifade eder.

Kabul edilen cevaba göre. Evet, "vekil" daha gevşek bir tanımlamaya sahiptir ve bir takımın ihtiyacına göre farklı şekilde kullanılabilir.

8 geliştirici ekibimizde, çoğu PR'da, değişiklikleri öneren ve nihayetinde PR'yi onaylayan 1 inceleme uzmanımız var. İnceleme aşamasında "atanan kişi" PR'ı açan kişidir; daha sonra PR başka geliştiriciler tarafından alınırsa, yeni bir "atanan kişi" eklenir. Halkla İlişkiler onaylandıktan ve KG veya doğrudan birleştirmeye hazır olduktan sonra, yeni bir KG "vekili" eklenir. Bu şekilde "vekil" listesi büyür.

Aşağıdaki kişileri toplu olarak belirlemek için "vekil" kullanıyoruz:

1. Çekme İsteği Yazarı
2. Halkla ilişkiler değişikliği önerileri üzerinde çalışan yazar (genellikle 1 ile aynı)
3. İlgili KG kişisi
4. Birleştirmeden sorumlu kişi (genellikle 2 veya 3 ile aynı)

"Vekil" kullanmak, gelecekte PR'nin kolayca bulunmasına yardımcı olur. Projemden birinde 3000'den fazla PR var.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Veya yalnızca author:raya-dumasyazarın oluşturduğu tüm öğeleri (sorunlar, PR'ler) bulmak için

ve arama işlemini kolaylaştırmak için diğer benzer sorgular. "kilometre taşları" PR aramayı kolaylaştırmak için de oldukça faydalıdır.

GitHub'dan önce sadece bir vekil alanı vardı ve inceleme alanı yoktu. O zamanlar hiçbir ayrım yoktu, bu nedenle vekil alanı en çok gözden geçirme alanı olarak kullanıldı.

Ancak projenize uygun şekilde kullanın.