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.

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.