Na GitHub, jaka jest różnica między recenzentem a cesjonariuszem?

186

Funkcja dodana 7 grudnia 2016 r., Ogłoszona na blogu GitHub, wprowadziła opcję dodawania recenzentów do żądania ściągnięcia

Opcja przeglądu GitHub

Możesz teraz wyraźnie poprosić współpracowników o sprawdzenie, co ułatwia określenie, komu chcesz przejrzeć prośbę o pobranie.

Możesz również zobaczyć listę osób, od których oczekujesz recenzji, na pasku bocznym strony żądania ściągnięcia, a także status recenzji od osób, które już je opuściły.

Jednak wyraźne ustawienie recenzenta dla PR zostało już wykonane poprzez przypisanie osób ( opcja cesjonariuszy ).

Kiedy obie opcje są już dostępne, jaka jest rola każdej opcji, ponieważ obie mają ten sam cel końcowy?

Cezar Augusto
źródło
1
kiedy po raz pierwszy pojawia się „funkcja przypisana”? Czy jest jakiś artykuł, który to wprowadza?
babeyh

Odpowiedzi:

134

EDYTOWAĆ:

Po dyskusji z kilkoma opiekunami OSS, recenzenci są definiowani jako słowo, które powinno być: recenzowanie (czyjś kod), a „cesjonariusz” ma luźniejszą definicję wyjaśnioną poniżej.

Dla „recenzenta” : ktoś, kogo chcesz sprawdzić kod. Niekoniecznie osoba odpowiedzialna za ten obszar lub odpowiedzialna za scalenie zatwierdzenia. Może to być ktoś, kto pracował wcześniej nad tym fragmentem kodu, jak sugeruje auto GitHub.

Dla „cesjonariusza” : do zespołu / opiekuna projektu, co to znaczy i nie ma ścisłej definicji. Może to być otwieracz do PR lub osoba odpowiedzialna za ten obszar (kto przyjmie PR po zakończeniu przeglądu lub po prostu go zamknie). GitHub nie musi definiować, co pozostawia otwarte dla opiekunów projektów, co najlepiej pasuje do ich projektu.

Poprzednia odpowiedź:

Ok, pójdę naprzód i odpowiem na własne pytanie.

Dla PR użytkowników z dostępem do zapisu: Cesjonariuszem byłaby ta sama osoba, która otworzyła PR, a recenzent zastąpiłby starą funkcję cesjonariusza (kod recenzji), będąc osobą wybraną przez cesjonariusza.

W przypadku PR użytkowników bez dostępu do zapisu (spoza współautorów): Ktoś z dostępem do zapisu wyznaczy siebie (lub innego członka z uprawnieniami do zapisu) do przeglądu PR (Recenzent). Cesjonariusz jest pusty.

W przypadku niedokończonego PR od zewnętrznych współpracowników : członek z prawem do zapisu zabrałby niedokończoną pracę i przypisał jej. Będzie odpowiedzialna za ukończenie zadania, będąc Cesjonariuszem . Ponieważ głównym powodem PR jest przeglądanie zmian, wybrałaby inne osoby do przejrzenia zmian.

Cezar Augusto
źródło
24
Do każdego nowego członka zespołu powinienem wysłać link do tej odpowiedzi, aby wyjaśnić, jak postępować z cesjonariuszami i recenzentami. Co prowadzi mnie do myśli, że coś tu jest zasadniczo nie tak :(
Andrey Kuleshov
Czy cesjonariusz musi mieć dostęp do zapisu?
Emre Sülün
czy istnieje różnica w zachowaniu powiadomień e-mail między nimi?
jxramos
26

W GitHub recenzent to osoba, która przegląda żądanie ściągnięcia. Właściciel projektu może zażądać przeglądu od dowolnego opiekuna, może nawet ustawić opcję, aby żądanie ściągnięcia mogło zostać scalone tylko wtedy, gdy zostanie przejrzane przez jednego z opiekunów z dostępem do zapisu.

Według urzędnika dokumentacją github , Cesjonariusz jest osobą, która pracuje nad konkretnymi problemami i wyciąga wnioski. Czasami jest mylony jako recenzent. W rzeczywistości jest przeznaczony do użycia z problemami, a nie do ściągania żądania, dzięki czemu po otrzymaniu problemu możemy wyznaczyć kogoś do jego rozwiązania. W żądaniu ściągnięcia cesjonariusz odnosi się do osoby, która jest odpowiedzialna za scalenie tego żądania ściągnięcia po otrzymaniu komentarzy i zmian wniosków od innych opiekunów.

Gautam Krishna R.
źródło
2
Dziękuję za odpowiedź, ale nie sądzę, że w pełni odpowiada na pytanie. Możesz przypisać komuś problem (więc ona będzie cesjonariuszem problemu), ale kiedy PR zostanie wysłany, ktoś będzie recenzentem (cesjonariuszem PR), a na tym etapie nadal nie jestem pewien, jaka jest różnica między cesjonariuszem a recenzent.
Cezar Augusto
14

Zgodnie z przyjętą odpowiedzią. Tak, „cesjonariusz” ma luźniejszą definicję i może być różnie wykorzystywany w zależności od potrzeb zespołu.

W naszym zespole 8 deweloperów, w większości PR mamy 1 recenzenta, który sugeruje zmiany i ostatecznie zatwierdza PR. W fazie przeglądu „cesjonariuszem” jest osoba, która otworzyła PR; później, jeśli PR zostanie odebrany przez innego programistę, dodaje się nowego „cesjonariusza”. Po zatwierdzeniu PR i przygotowaniu go do kontroli jakości lub bezpośredniego scalenia dodaje się nowego „cesjonariusza” kontroli jakości. W ten sposób rośnie lista „cesjonariuszy”.

Używamy „cesjonariusza” do wspólnego wyznaczania następujących osób:

  1. Pull Request Author
  2. Autor pracujący nad sugestiami zmiany PR (zwykle taki sam jak 1)
  3. Zaangażowana osoba QA
  4. Osoba odpowiedzialna za scalenie (zwykle taka sama jak 2 lub 3)

Użycie „cesjonariusza” pomaga łatwo zlokalizować PR w przyszłości. Jeden z moich projektów ma> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Lub tylko author:raya-dumas znaleźć wszystkie elementy utworzone przez autora (problemy, PR)

i inne podobne zapytania, aby ułatwić proces wyszukiwania. „kamieni milowych” są również bardzo przydatne w celu ułatwienia wyszukiwania PR.

Screenshot Github, IV kwartał 2017 r

lal
źródło
Bardzo dobrze wyjaśnione.
Nitin Gaur
Należy wspomnieć, że możesz po prostu poszukać autora: my-github-handle, aby znaleźć to, co PR stworzył osoba
Wisienkas
1

Przed GitHub miał tylko pole cesjonariusza i brak pola recenzenta. Nie było wtedy rozróżnienia, więc pole cesjonariusza było najczęściej używane jako pole recenzenta.

Używaj ich jednak w dowolny sposób.

ericcurtin
źródło