Uzyskiwanie opinii za pomocą żądań ściągnięcia

Żądania ściągnięcia obsługują przeglądanie i scalanie kodu w jednym procesie współpracy. Po dodaniu przez dewelopera funkcji lub poprawki błędów tworzą żądanie ściągnięcia, aby rozpocząć proces scalania zmian z gałęzią nadrzędną. Inni członkowie zespołu mają szansę przejrzeć i zatwierdzić kod przed jego sfinalizowanie. Użyj żądań ściągnięcia, aby przejrzeć prace w toku i uzyskać wczesną opinię na temat zmian. Nie ma jednak zobowiązania do scalania zmian. Właściciel może w dowolnym momencie porzucić żądanie ściągnięcia.

Pobieranie przeglądu kodu

Przegląd kodu wykonany w ramach żądania ściągnięcia nie jest tylko do znajdowania oczywistych usterek; to właśnie są testy. Dobry przegląd kodu przechwytuje mniej oczywiste problemy, które mogą prowadzić do kosztowych problemów później.

Przeglądy kodu pomagają chronić zespół przed nieprawidłowymi scalaniami i uszkodzonymi kompilacjami, które zwiększają produktywność zespołu. Przeglądy przechwytują problemy przed scalaniem, chroniąc ważne gałęzie przed niechcianymi zmianami.

Przeglądy kodu zachęcają również i wzmacniają współpracę i komunikację między deweloperami. A zespół zyskuje wyraźną historię wszystkich zmian wprowadzonych między gałęzią główną a gałęziami funkcji.

Doświadczenie w zakresie zapylania i rozpowszechnianie strategii rozwiązywania problemów przy użyciu szerokiej gamy recenzentów w przeglądach kodu. Różnice umiejętności i wiedza sprawiają, że zespół jest silniejszy i bardziej odporny.

Prześlij świetną opinię

Recenzje wysokiej jakości zaczynają się od wysokiej jakości opinii. Klucze do doskonałej opinii w żądaniu ściągnięcia to:

• Poproś odpowiednich użytkowników o przejrzenie żądania ściągnięcia.
• Upewnij się, że recenzenci wiedzą, co robi kod.
• Przekaż konstruktywne informacje zwrotne z możliwością działania.
• Odpowiedz na komentarze w odpowiednim czasie.

Po przypisaniu recenzentów do żądania ściągnięcia należy wybrać odpowiedni zestaw recenzentów. Recenzenci powinni wiedzieć, jak działa kod, ale także deweloperzy pracujący w innych obszarach, aby mogli podzielić się swoimi pomysłami.

Podaj jasny opis zmian i podaj kompilację kodu, który zawiera poprawkę lub funkcję działającą w nim. Recenzenci powinni starać się przekazać opinię na temat zmian, z którymi nie zgadzają się. Zidentyfikuj problem i podaj konkretne sugestie dotyczące tego, co można zrobić inaczej. Ta opinia ma wyraźną intencję i jest łatwa dla właściciela żądania ściągnięcia, aby zrozumieć.

Właściciel żądania ściągnięcia powinien odpowiedzieć na komentarze, zaakceptować sugestie lub wyjaśnić, dlaczego odrzucają ich stosowanie. Niektóre sugestie są dobre, ale mogą znajdować się poza zakresem żądania ściągnięcia. Weź te sugestie i utwórz nowe elementy robocze i gałęzie funkcji niezależnie od żądania ściągnięcia, aby wprowadzić te zmiany.

Ochrona gałęzi za pomocą zasad

Istnieje kilka krytycznych gałęzi w repozytorium, które zespoły polegają na zawsze w dobrej formie, takiej jak main gałąź. Zespoły mogą wymagać żądań ściągnięcia, aby wprowadzić wszelkie zmiany w tych gałęziach za pomocą platform takich jak GitHub i Azure DevOps. Deweloperzy wypychając zmiany bezpośrednio do chronionych gałęzi będą mieli odrzucone wypychania.

Dodaj dodatkowe warunki do żądań ściągnięcia, aby wymusić wyższy poziom jakości kodu w kluczowych gałęziach. Czysta kompilacja scalonego kodu i zatwierdzenia przez wielu recenzentów to pewne dodatkowe wymagania, które są często stosowane w celu ochrony kluczowych gałęzi.

Dowiedz się więcej

Usługa GitHub zawiera obszerną dokumentację dotyczącą sposobu zaproponowania zmian w pracy z żądaniami ściągnięcia.

Przeczytaj więcej na temat przekazywania wspaniałych opinii w przeglądach kodu i używania szablonów żądań ściągnięcia w celu udostępnienia wskazówek recenzentom. Usługa Azure DevOps oferuje również zaawansowane środowisko żądań ściągnięcia, które jest łatwe w użyciu i skaluje w razie potrzeby.