Jak przeprowadzać recenzje użytkowników na temat żądań ściągania GitHub?

12

Przechodzimy z Bitbucket do GitHub, a jedną rzeczą, z którą mamy problemy, są recenzje kodów równorzędnych, które działały bardzo płynnie na Bitbucket w następujący sposób:

  1. Autor otworzył żądanie ściągnięcia (GitHub: to samo)
  2. Autor dodał swoich współpracowników jako recenzentów (GitHub:? Walczy tu z wieloma osobami przypisanymi)
  3. Recenzent:
    1. Zatwierdzono PR z zielonym znacznikiem wyboru (GitHub: ??)
    2. Dodano komentarze (GitHub: to samo)
    3. Utworzono lekkie zadania (GitHub: coś podobnego, jeśli - [ ]w opisie PR jest używana składnia; szkoda, że ​​nie działa dla zadań)
  4. Jest lista PR, które mogę zobaczyć na pierwszy rzut oka, które są sprawdzone i OK do połączenia i które wymagają dalszej uwagi (GitHub: ??)

Powinienem zaznaczyć, że chcemy unikać narzędzi do sprawdzania kodu innych firm, jeśli to w ogóle możliwe, i chcielibyśmy pozostać na waniliowym GitHubie z pewnymi rozwiązaniami.

Borek Bernard
źródło
1
Wygląda na to, że przełączasz się przedwcześnie. Po co zrezygnować, zwłaszcza jeśli nowa rzecz nie ma wszystkich potrzebnych funkcji?
niania
Napisz komentarz do swojego prq i zaznacz @ ktokolwiek chcesz otrzymać powiadomienie. Recenzent może dodawać tagi, aby pokazać swoją opinię.
Wilbert
Pamiętaj też, że możesz @mentionować cały zespół, np. @ MyCompany / developers
Whymarrh

Odpowiedzi:

6

Z tego, co widziałem, większość tych kroków jest wykonywana w Github na podstawie konwencji, a nie przez żaden oficjalny proces dostarczony przez Github.

Mój pracodawca korzysta z Github, prowadzę wiele małych projektów open source i od czasu do czasu wnoszę wkład w inne projekty open source.

Oto jak zwykle to widziałem:

Autor dodając swoich współpracowników jako recenzentów:

Różni się to w zależności od projektu, ale ogólnie rzecz biorąc, wszyscy przypisani recenzenci są wszystkimi uczestnikami projektu .

Wydaje się, że projekty typu open source mają zgrubną hierarchię - być może ich konwencja polegałaby na łączeniu się tylko wtedy, gdy „główny” współpracownik wyrazi zgodę

W sklepie, w którym obecnie jestem zatrudniony, łączymy się po zatwierdzeniu przez jednego z kilkudziesięciu programistów w zespole.

W rzadkich przypadkach ktoś w zespole może użyć komentarza, aby konkretnie wezwać innego programistę, który według niego powinien przejrzeć kod przed jego scaleniem, ale w przeciwnym razie każdy, kto się tam pojawi i ma na to ochotę, może przeglądać i komentować.

Zatwierdzenie recenzenta:

Zatwierdzenie jest zwykle pokazywane w komentarzu do żądania ściągnięcia z napisem „+1” lub „lgtm” (dla mnie wygląda dobrze).

Lekkie zadania:

Użyłem również pól wyboru, ale w większości przypadków każdy komentarz do żądania ściągnięcia jest uważany za dorozumiane „zadanie”, które można rozwiązać poprzez:

  • zmiana kodu komentowanego przez linię
  • odpowiadanie innym komentarzem

Zobacz, co jest zatwierdzone, a co nadal wymaga przeglądu:

Użyłem rozszerzenia Looks Good To Me dla Chrome, które daje taki widok z ekranu Pull Requests. Wygląda na to, że widok listy żądań ściągnięcia został uszkodzony przez ostatnie zmiany w Github.

TehShrike
źródło