2016-12-11 41 views
74

Cechą Dodany gru, 7, 2016, ogłosił na GitHub blogu, wprowadzono opcję to add reviewers to a Pull RequestNa GitHub, jaka jest różnica między recenzentem a cesjonariuszem?

GitHub Review Option

Teraz możesz poprosić o sprawdzenie wyraźnie od współpracowników, dzięki czemu łatwiej określić, kim jesteś” Chciałbym przejrzeć Twoją prośbę o odciągnięcie.

Możesz również wyświetlić listę osób, na które oczekujesz przeglądu, na pasku bocznym strony żądania pobrania, a także status opinii od tych, którzy już je opuścili.

jednak jednoznaczne ustawienie recenzentem dla PR zostało już zrobione przez przypisanie osób (nabywców praw opcja).

Przy obu dostępnych opcjach, jaka jest rola każdej opcji, ponieważ oba mają ten sam cel końcowy?

Odpowiedz

63

EDIT:

Po przedyskutowaniu z kilkoma OSS opiekunów recenzentów jest zdefiniowane jako co to słowo powinno być: do przeglądu (czyjś kod) oraz „cesjonariusz” ma luźniejsze Określenie to wyjaśnione poniżej .

Dla "recenzenta": ktoś, kogo chcesz przejrzeć kod. Niekoniecznie osoba odpowiedzialna za ten obszar lub odpowiedzialna za scalenie zatwierdzenia. Może to być ktoś, kto pracował nad tym kawałkiem kodu, jak to sugeruje GitHub automatycznie.

Dla "osoby cesjonariusza": do zespołu/opiekuna projektu, co to oznacza i nie ma ścisłej definicji. Może być otwieraczem PR lub osobą odpowiedzialną za ten obszar (kto zaakceptuje PR po zakończeniu przeglądu lub po prostu go zamknie). To nie GitHub definiuje to, co pozostawia otwartą dla opiekunów projektu, co najlepiej pasuje do ich projektu.

Poprzednia odpowiedź:

Ok będę śmiało odpowiedzieć na moje własne pytanie.

PR użytkowników z write-access: cesjonariusza byłaby ta sama osoba, która otworzyła Pr, a Recenzent zastąpi (kod reviewing) stare funkcja cesjonariusz, jest to jeden z cesjonariuszem ktoś wyboru.

PR użytkowników bez write-access (współtwórcy zewnątrz): Ktoś z write-access by przypisać sobie (lub inny element zapisu przywilej), aby przejrzeć PR (recenzent). Cesjonariusz jest pusty.

Za niedokończone PR od zewnętrznych współpracowników: członek dostępu do zapisu wziąłby niedokończoną pracę i wyznaczył ją. Ona będzie odpowiedzialna za ukończenie zadania, będąc Assignee. Ponieważ głównym powodem PR jest przegląd zmian, wybrałaby kilka osób do przeglądu zmian.

+6

Dla każdego nowego członka zespołu powinienem wysłać link do tej odpowiedzi, aby wyjaśnić, jak radzić sobie z cesjonariuszami i recenzentami. Co prowadzi mnie do przekonania, że ​​coś tu jest z gruntu nie tak :( –

12

W serwisie GitHub recenzent to osoba, która sprawdza żądanie pobrania. Właściciel projektu może zażądać recenzji od dowolnego opiekuna, może nawet ustawić opcję, aby żądanie pobrania mogło zostać scalone tylko wtedy, gdy zostanie sprawdzone przez jednego z opiekunów z prawem do zapisu.

Zgodnie z oficjalnym github documentation, Cesarz jest osobą, która pracuje nad konkretnymi sprawami i wnioskami o wycofanie. Czasami jest mylony jako recenzent. W rzeczywistości ma być używany z problemami, a nie z prośbą o ściągnięcie, więc gdy otrzymamy problem, możemy wyznaczyć kogoś, kto to naprawi. W przypadku żądania pobrania cesjonariusz odnosi się do osoby, która jest odpowiedzialna za scalenie tego żądania pobrania po otrzymaniu komentarzy i zmianie żądań od innych opiekunów.

+2

Dziękuję za odpowiedź, ale nie sądzę, że w pełni odpowiada na to pytanie. Możesz przypisać komuś problem (będzie to więc sprawa cesjonariusza), ale kiedy zostanie wysłany PR, ktoś będzie recenzentem (cesjonariuszem PR), i na tym etapie wciąż nie jestem pewien różnicy między cesjonariuszem a recenzent. –

3

Zgodnie z zaakceptowaną odpowiedzią. Tak, "cesjonariusz" ma luźniejszą definicję i może być używany w inny sposób do potrzeb zespołów.

W naszym zespole 8 developerów, w większości PR-ów mamy 1 recenzenta, który sugeruje zmiany i ostatecznie zatwierdza PR. W fazie przeglądu "cesjonariusz" jest osobą, która otworzyła PR; później, jeśli PR zostanie odebrany przez innego programistę, zostanie dodany nowy "cesjonariusz". Po zatwierdzeniu PR i przygotowaniu do QA lub bezpośredniego scalenia, nowy QA "cesjonariusz" zostaje dodany. W ten sposób rośnie lista "cesjonariusza".

Używamy "cesjonariusza", aby wyznaczyć następujące osoby zbiorowo:

  1. Pull Zapytanie Autor
  2. Autor pracuje nad propozycjami zmian PR (zazwyczaj taki sam jak 1)
  3. QA osoba zaangażowana
  4. Osoba odpowiedzialna do scalenia (zwykle to samo co 2 lub 3)

Korzystanie z "cesjonariusza" pomaga w lokalizacji PR w przyszłych programach ily. Jeden z moich projektów ma> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

i inne podobne pytania, aby ułatwić proces wyszukiwania. "kamienie milowe" są bardzo przydatne, aby ułatwić wyszukiwanie PR.

Screenshot Github, Q4 2017