Udostępnij przez


Informacje o żądaniach ściągnięcia

Azure DevOps Services | Azure DevOps Server | Azure DevOps Server 2022

Żądania ściągnięcia to sposób zmiany, przeglądania i scalania kodu w repozytorium Git w usłudze Azure Repos. Żądania ściągnięcia mogą pochodzić z gałęzi w tym samym repozytorium lub z gałęzi w rozwidleniu repozytorium. Zespoły używają pull requestów do przeglądania kodu i przekazywania opinii na temat zmian przed scaleniem kodu z gałęzią główną. Recenzenci mogą przechodzić przez proponowane zmiany, pozostawiać komentarze i głosować, aby zatwierdzić lub odrzucić kod.

W tym artykule opisano wytyczne dotyczące pull requestów i kwestie związane z zarządzaniem. Aby uzyskać instrukcje dotyczące tworzenia, wyświetlania, przeglądania i wykonywania żądań ściągnięcia, zobacz następujące artykuły:

Uwaga

Ze względów wydajnościowych i stabilności liczba recenzentów, których można dodać do pull requestu, musi wynosić 1000 lub mniej. Nowe żądania ściągnięcia nie zostaną utworzone podczas dodawania więcej niż 1000 recenzentów, a istniejące żądania ściągnięcia nie umożliwią dodania więcej niż 1000 recenzentów.

Uprawnienia i wymagania wstępne

  • Repozytoria muszą być włączone w twoim projekcie. Jeśli centrum repozytoriów i skojarzone strony nie są wyświetlane, zobacz Włączanie lub wyłączanie usługi Azure DevOps w celu ponownego włączenia repozytoriów.

  • Aby wyświetlić lub przejrzeć żądania wprowadzenia zmian (PR), należy być członkiem projektu Azure DevOps z co najmniej podstawowym dostępem.

  • Aby wnosić wkład do PR, być członkiem grupy zabezpieczeń Czytelnicy lub mieć odpowiednie uprawnienia.

  • Aby utworzyć i ukończyć żądanie włączenia zmian, należy być członkiem grupy zabezpieczeń współautorów lub posiadać odpowiednie uprawnienia.

Uwaga

W przypadku projektów publicznych użytkownicy, którym udzielono dostępu uczestnikom projektu , mają pełny dostęp do usługi Azure Repos.

  • Repozytoria muszą być włączone w twoim projekcie. Jeśli centrum repozytoriów i skojarzone strony nie są wyświetlane, zobacz Włączanie lub wyłączanie usługi Azure DevOps w celu ponownego włączenia repozytoriów.
  • Aby wyświetlić lub przejrzeć żądania wprowadzenia zmian (PR), należy być członkiem projektu Azure DevOps z co najmniej podstawowym dostępem. Jeśli nie jesteś członkiem projektu, dodaj go.
  • Aby wnosić wkład do PR, być członkiem grupy zabezpieczeń Czytelnicy lub mieć odpowiednie uprawnienia.
  • Aby utworzyć i ukończyć żądanie włączenia zmian, należy być członkiem grupy zabezpieczeń współautorów lub posiadać odpowiednie uprawnienia.

Aby uzyskać więcej informacji o uprawnieniach i dostępie, zobacz Domyślne repozytorium Git i uprawnienia gałęzi oraz Informacje o poziomach dostępu.

Opinie dotyczące jakości pull requestów

Recenzje wysokiej jakości zaczynają się od wysokiej jakości opinii. Oto kilka wskazówek do skutecznego feedbacku dotyczącego PR:

  • Właściciel pull requesta powinien zadbać o to, aby odpowiednie osoby dokonały przeglądu i upewnił się, że recenzenci wiedzą, co robi kod.
  • Recenzenci powinni udzielić konstruktywnej, możliwej do wdrożenia opinii.
  • Właściciele i recenzenci powinni szybko komentować i odpowiadać.

Właściciele pull requestów powinni:

  • Pamiętaj, aby wybrać odpowiednich recenzentów do przypisania do PR.
  • Uwzględnij recenzentów, którzy wiedzą, jak działa kod.
  • Poproś deweloperów pracujących w innych obszarach, aby podzielili się swoimi pomysłami.
  • Podaj jasny opis zmian.
  • Udostępnij recenzentom wskazówki przy użyciu szablonów próśb o dołączenie.
  • Podaj kompilację kodu z uruchomioną poprawką lub funkcją.
  • Odpowiedz na komentarze, akceptując sugestię lub wyjaśniając, dlaczego sugerowana zmiana nie jest idealna.
  • Dla dobrych sugestii poza zakresem PR, utwórz nowe elementy robocze, gałęzie i pull requesty, aby wprowadzić te zmiany.

Recenzenci powinni wykonywać następujące zadania.

  • Prześlij opinię na temat zmian, z którymi nie zgadzają się
  • Identyfikowanie problemów i nadawanie konkretnych sugestii dotyczących tego, co należy zrobić inaczej
  • Upewnij się, że opinia ma wyraźną intencję i jest łatwa do zrozumienia
  • Pozostawianie komentarzy lub głosowanie nad zmianami

Aby uzyskać więcej informacji, zobacz Uzyskiwanie opinii za pomocą żądań ściągnięcia w Git.

Polityki gałęzi i pull requesty

Twój zespół może polegać na gałęziach krytycznych w repozytorium, takich jak main gałąź, aby zawsze mieć dobrą formę. Można ustawić polityki gałęzi, aby wymagać żądań ściągnięcia dla każdej zmiany w tych chronionych gałęziach i odrzucić wszelkie zmiany przesłane bezpośrednio do gałęzi.

Możesz dodać więcej polityk do Pull Requestów, aby wymusić lepszą jakość kodu w kluczowych gałęziach. Dodatkowe wymagania, takie jak czysta kompilacja proponowanego kodu lub zatwierdzenia przez wielu recenzentów, mogą pomóc w ochronie kluczowych gałęzi.

Liczbę wymaganych zatwierdzeń dla pull requesta można ustawić w zasadach gałęzi. Można również ustawić, że niektórzy recenzenci mają być wymagane lub opcjonalne dla wszystkich lub niektórych żądania ściągnięcia. Żądanie ściągnięcia (pull request) można ustawić na automatyczne uzupełnianie z wymaganą liczbą zatwierdzeń, nawet jeśli inni recenzenci odrzucą zmiany. Jednak wymagani recenzenci muszą zatwierdzić PR-y, zanim PR-y będą mogły zostać scalone. Zaleca się, aby zmiany w znaczącym pull request były przeglądane i zatwierdzane przez co najmniej dwóch recenzentów.

Aby zresetować głosy za każdym razem, gdy autor żądania ściągnięcia wprowadza nowe zmiany, wybierz pozycję Resetuj głosy recenzentów, gdy są nowe zmiany w polityce gałęzi Wymagaj minimalnej liczby recenzentów.

Poniższa tabela zawiera podsumowanie zasad, które można zdefiniować w celu dostosowania gałęzi. Aby zapoznać się z omówieniem wszystkich zasad i ustawień repozytorium oraz gałęzi, zobacz Ustawienia i zasady repozytorium Git.

Zasady

Wartość domyślna

Opis


Wyłączony

Wymagaj zatwierdzenia od określonej liczby recenzentów pull requestów.

Wyłączony

Zachęcaj do śledzenia, sprawdzając połączone elementy robocze w żądaniach ściągnięcia

Wyłączony

Sprawdź, czy wszystkie komentarze w pull requestach zostały rozwiązane.

Wyłączony

Kontrolowanie historii gałęzi poprzez ograniczenie dostępnych typów scalania przy finalizacji żądań pull.

Wyłączony

Dodaj jedne lub więcej zasad, aby zweryfikować kod przed scaleniem i stworzeniem zmian w żądaniu wciągnięcia. Może również włączać lub wyłączać zasady.

Wyłączony

Dodaj co najmniej jedną zasadę, aby inne usługi publikowały pomyślny status, aby ukończyć żądania ściągnięcia. Może również włączać lub wyłączać zasady.

Wyłączony

Dodaj jedną lub więcej zasad, aby wyznaczyć recenzentów kodu, którzy będą automatycznie uwzględniani, gdy zgłoszenia zmian obejmują określone obszary kodu. Może również włączać lub wyłączać zasady.

Aby uzyskać więcej informacji, zobacz:

Definiowanie kontroli stanu w celu poprawy jakości kodu

Żądania wciągnięcia i zasady gałęzi umożliwiają zespołom egzekwowanie najlepszych praktyk dotyczących przeglądania kodu i uruchamiania automatycznych kompilacji. Wiele zespołów ma dodatkowe wymagania i walidacje, które należy wykonać w kodzie. Aby uwzględnić te potrzeby, możesz zintegrować kontrole stanu PR z przepływem pracy PR. W przypadku sprawdzania statusu pull request, usługi zewnętrzne mogą programowo zatwierdzać zmiany w kodzie, kojarząc informacje o powodzeniu lub niepowodzeniu z pull request.

Aby uzyskać więcej informacji, zobacz następujące artykuły:

Problem z wieloma bazami scalania

W niektórych przypadkach PR ma więcej niż jeden prawdziwy punkt początkowy scalania, a taka sytuacja może prowadzić do problemów z bezpieczeństwem. Jeśli pliki w pull request mają różne wersje w różnych bazach scalania, pojawia się ostrzeżenie o wielu bazach scalania. Aby uzyskać więcej informacji i korekty, zobacz Wiele baz scalania.

Następne kroki