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:
- Autor otworzył żądanie ściągnięcia (GitHub: to samo)
- Autor dodał swoich współpracowników jako recenzentów (GitHub:? Walczy tu z wieloma osobami przypisanymi)
- Recenzent:
- Zatwierdzono PR z zielonym znacznikiem wyboru (GitHub: ??)
- Dodano komentarze (GitHub: to samo)
- Utworzono lekkie zadania (GitHub: coś podobnego, jeśli
- [ ]
w opisie PR jest używana składnia; szkoda, że nie działa dla zadań)
- 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.
code-reviews
github
pull-requests
Borek Bernard
źródło
źródło
Odpowiedzi:
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:
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.
źródło