Granska pull-begäranden

Azure DevOps Services | Azure DevOps Server 2022 – Azure DevOps Server 2019

När du skapar en pull-begäran (PR) och anger nödvändiga/valfria granskare meddelar Azure Repos granskarna att din pr är redo för granskning. Alla nödvändiga granskare måste godkänna ändringarna i din PR innan ändringarna kan sammanfogas till målgrenen. För PR:er som ändrar viktiga grenar som grenen main kan ditt team ha skapat grenprinciper som anger granskarna eller kräver ett minsta antal granskare. Om grenprinciper lägger till valfria granskare i din pr kan du behålla dem, kräva dem eller ta bort dem. Om grenprinciper lägger till nödvändiga granskare i din pr kan du inte göra dem valfria eller ta bort dem. Information om hur du tilldelar granskare via grenprinciper finns i Inkludera kodgranskare automatiskt.

Om du granskar en pr kan du försöka ge konstruktiv feedback som är exakt och lätt att förstå. För mer information om granskningsfeedback, se Feedback om pull-begäranden. Den här artikeln beskriver hur du granskar pull-begäranden i Azure DevOps.

Du kan bara granska Azure DevOps-PR:er i webbportalen med hjälp av webbläsaren.

I den här artikeln lär du dig hur du:

• Granska ändringar
• Använda kommentarer
• Redigera filer
• Spåra granskade filer
• Rösta om PR-ändringar
• Slutför en PR

Förutsättningar

• Arkiv måste vara aktiverade i dina Azure DevOps-projektinställningar. Om Repos-hubben och de associerade sidorna inte visas kan du läsa Aktivera eller inaktivera en Azure DevOps-tjänst för att återaktivera Repos.

• För att granska PR:n måste du vara medlem i det Azure DevOps-projekt där PR:n finns, med åtminstone Grundläggande åtkomst.

  • Om du inte har ett projekt skapar du ett eller registrerar dig kostnadsfritt.

  • Om du inte är projektmedlem läggs du till.

  Anteckning

  För offentliga projekt har användare som beviljats intressentåtkomst fullständig åtkomst till Azure Repos.

• Mer information om behörigheter och åtkomst finns i Git-standardlagringsplats och grenbehörigheter och Om åtkomstnivåer.

Du kan hantera PR:er i Azure DevOps Services med hjälp av Kommandoradsgränssnittet för Azure DevOps (CLI). Azure DevOps CLI är tillgängligt när du installerar Azure CLI och Azure DevOps-tillägget. En lista över CLI-kommandon för att hantera PR finns i az repos pr kommandona.

Granska ändringar

Pull requests låter utsedda granskare undersöka, diskutera och rösta på föreslagna ändringar innan ändringarna tillämpas på en målgren i ett repository. Tänk på att om utvecklingen utförs med samma filer på olika grenar kan det uppstå motstridiga ändringar. I så fall krävs det att du noggrant analyserar och löser dessa konflikter. För att göra det kan det kräva att filen redigeras manuellt för att korrekt tillämpa ändringarna från antingen källa, mål eller båda grenarna. Att ta alla ändringar från antingen källa eller mål (vilket kan göras med ett enda knappklick) kanske inte är optimalt och kan leda till förlust av vissa nödvändiga ändringar. Detta kan göras med hjälp av ADO-verktyg, eller så är det enklare att använda utvecklingsverktyg (till exempel Visual Studio).

Mer information finns i Lösa sammanslagningskonflikter.

Följande steg beskriver hur granskare av pr i Azure Repos kan navigera genom en pr för att förstå de föreslagna ändringarna:

1. Öppna teamprojektet för din Azure DevOps-organisation från webbläsaren. Välj Repo > Pull-begäranden för att visa en lista över pr-begäranden. En nyligen öppnad PR är förvald till fliken Översikt.

2. Fliken Översikt för en PR visar rubrik, beskrivning, granskare, länkade arbetsobjekt, historik, status och kommentarer. Läs PR-beskrivningen för att se de föreslagna ändringarna. Visa kommentarerna för att förstå de problem som andra granskare har tagit upp.

   Kommentar

   Varje fil uppskattas för ett antal ändringar och markeras med ett "+"-tecken eller en "byt namn, redigera"-etikett bredvid dess namn. När en fil har fler än 50 procent ändringar anses den ha bytt namn. Det här är standardtröskelvärdet för git-lagringsplatser som inte kan ändras.

   

3. Välj fliken Filer för att granska alla innehållsändringar i PR:s källgren. Den första vyn visar en sammanfattningsvy över alla filändringar. Välj knappen Visa bredvid en fil för att endast visa filens ändringar. Om filen har ändrats öppnar knappen Visa en diffvy. Om filen har lagts till eller tagits bort öppnar knappen Visa ett innehållsfönster.

   

   Kommentar

   Av prestandaskäl visar sammanfattningsvyn inte ändringar för en fil som är större än 0,5 MB.

4. I en diff-vy för en fil kan du välja antingen en sid-vid-sida - eller infogad difflayout.

   

   Tips

   För en enskild fil som är större än 5 MB visar diffvyn trunkerat filinnehåll. För en diff-vy över det fullständiga innehållet laddar du ned och granskar sådana filer med hjälp av ett lokalt diff-verktyg.

   Använd det vänstra filträdet för att visa en annan fil eller välj roten i filträdet för att se en sammanfattningsvy över alla filändringar.

5. Om du vill granska den ändringsuppsättning som introduceras av specifika push-meddelanden till källgrenen väljer du en eller flera ändringsuppsättningar i listrutan ändringar. När du väljer en eller flera ändringsuppsättningar uppdateras diffvyn så att endast ändringarna från de valda ändringsuppsättningarna visas. Den här funktionen är användbar när ändringar har pushats till PR sedan din senaste granskning och du bara vill se de senaste ändringarna. Listrutan ändringar namnger varje ändringsuppsättning med incheckningsmeddelandet från den slutliga incheckningen i varje push-åtgärd.

   

   Tips

   Håll ned Skift-tangenten när du väljer flera ändringsuppsättningar i listrutan ändringar.

6. Välj fliken Uppdateringar för att visa alla push-överförda ändringsuppsättningar för att säkerställa att du inte har missat några ändringar i källgrenen. Ändringarna numreras och den senaste ändringsuppsättningen visas överst i listan. Varje ändringsuppsättning visar de commits som överfördes genom push i denna åtgärd. En forcerad ändringsuppsättning skriver inte över ändringshistoriken för ändringsuppsättningen och visas i listan över ändringsuppsättningar på samma sätt som vilken annan ändringsuppsättning som helst.

   

7. Välj fliken Incheckningar för att visa incheckningshistoriken för källgrenen efter att den avvek från målgrenen. Incheckningshistoriken på fliken Incheckningar skrivs över om PR-författaren framtvingar en annan incheckningshistorik, så incheckningarna som visas på fliken Incheckningar kan skilja sig från de incheckningar som visas på fliken Uppdateringar .

   

Granska filer

Välj fliken PR-filer för att visa de faktiska ändringar som gjorts i källgrenen bredvid målgrenen för pull-begäran.

Anteckning

De olika ändringarna mellan Azure DevOps pull-begäran och Azure DevOps förgreningsjämförelse beror på de olika jämförelsemetoderna.

Det finns två jämförelsemetoder för git diff-kommandot: Två punkter (git diff A..B) och tre punkter (git diff A...B). Som standard visar pull-begäranden ett trepunkts-diff medan grenjämförelse i stället visar ett tvåpunkts-diff.

Till exempel:

      C---D---E branch
     /
A---B---F---G   master

Pull-begäranden: git diff branch...master genererar endast C-, D-, E-kommandon. Branch Compare: git diff branch..master kommer att producera C, D, E, F, G incheckningar.

Mer information finns i trepunkt-och-tvåpunkt-git-diff-jämförelser och git diff

Anteckning

När du visar skillnaden för en enskild vald fil finns det en filstorleksgräns på 5 MB. Om du vill visa och diff-filer som är större än 5 MB kan du ladda ned filen och visa den med hjälp av ett lokalt diff-verktyg. När du visar skillnaden för en samling filer är storleksgränsen för varje fil 0,5 MB, av prestandaskäl.

Granska uppdateringar

Granska tidigare versioner av koden från listrutan Alla uppdateringar .

Varje uppdatering av grenen lägger till en ny version i listan och på fliken Uppdateringar i PR. När du väljer olika uppdateringar uppdateras diffvyn för att visa skillnaderna mellan filerna i varje version av PR.

Du kan komma ikapp med PR-uppdateringar efter att ha varit borta från PR genom att gå igenom ändringar som gjorts sedan din senaste granskning.

Bläddra i en lista över ändringar från författaren på fliken Uppdateringar .

Visa och välj ändringar som gjorts i incheckningar till grenen på fliken Incheckningar .

Använda kommentarer

PR-författare och granskare kan kommunicera med varandra genom att lägga till och svara på PR-kommentarer. När du granskar en pr använder du kommentarer för att peka på problem med de föreslagna ändringarna, föreslå ändringar och svara på tidigare kommentarer. Sträva efter konstruktiv feedback som är exakt och lätt att förstå. Adressera mottagare direkt med hjälp av deras @username. Referera till arbetsobjekt genom att använda #workitemID och andra PR:er genom att använda !pullrequestID. Ibland skapar PR-författare kommentarer för sig själva i dokumentationssyfte.

Ge feedback i kommentarer

Om din feedback gäller för en specifik rad eller radintervall i en fil lägger du till en kommentar på den platsen i filen:

1. Hovra över den rad som du vill kommentera på fliken Filer i en pr och välj kommentarsknappen för att öppna en infogad kommentarsruta. Du kan också välja flera rader och sedan välja kommentarsknappen som visas när du hovrar över dessa rader.

   

2. Ange din kommentar i kommentarsrutan och välj Kommentar.

   

Om din feedback gäller för hela filen lägger du till en kommentar på filnivå genom att välja Lägg till kommentar på menyn filalternativ:

Om du vill ge allmän feedback som inte är relaterad till en specifik fil lägger du till en kommentar på fliken Översikt :

PR-granskare kan också använda kommentarer för att föreslå ändringar av specifika filrader, enligt beskrivningen i nästa avsnitt.

Föreslå ändringar i kommentarer

Du kan använda kommentarer för att föreslå ersättningstext för en eller flera rader i en fil:

1. Hovra över den rad som du vill kommentera på fliken Filer i en pr och välj kommentarsknappen för att öppna en infogad kommentarsruta. Du kan också välja flera rader och sedan välja kommentarsknappen som visas när du hovrar över dessa rader. Om det godkänns ersätter det föreslagna innehållet raden eller raderna som kommentaren lades till i.

   

2. Välj glödlampaikonen under kommentarsrutan för att göra dina föreslagna ändringar i kommentarsrutan i ett inhägnat kodblock och välj sedan Kommentera. Du kommer inte att se en glödlampikon om du lägger till en kommentar på den ursprungliga koden (på vänster sida) i en diffvy sida vid sida.

   

PR-författare kan acceptera ditt förslag genom att:

1. Välj Tillämpa ändringar för att staga ändringen inför en incheckning.

   

2. Välj Genomför alla ändringar för att genomföra alla mellanlagrade ändringar. PR-författare kan avaktivera ett förslag genom att välja Ångra ändring.

   

Redigera eller ta bort kommentarer

Du kan redigera eller ta bort kommentaren genom att hovra över kommentaren och välja ikonen redigera eller ta bort :

Gilla kommentarer

PR-författare och granskare kan gilla sina egna eller någon annans kommentar genom att hovra över kommentaren och välja tummen upp-ikonen:

Kommentarer med gilla-markeringar visar en ifylld tum-upp-ikon och antalet gilla-markeringar. Hovra över tummen upp-ikonen för att se listan över personer som gillade kommentaren.

Svara på kommentarer

PR-författare bör svara på kommentarer för att låta granskarna veta hur de hanterar feedback och förslag:

1. Om du vill svara på en kommentar skriver du ditt svar i fältet Skriv ett svar . Adressera mottagare direkt med hjälp av deras @username. Referera till arbetsobjekt med hjälp av #workitemID och till andra PR:er med hjälp av !pullrequestID.

   

2. När du har angett svaret väljer du Svara och löser om svaret är slutgiltigt. Annars väljer du Svara.

   

   Om du väljer Svara och lös ändras kommentarsstatusen till Löst. PR-författare kan också direkt ändra status för en kommentar enligt beskrivningen i nästa avsnitt.

Ändra kommentarsstatus

Nya kommentarer har ursprungligen en aktiv status, som PR-författare uppdaterar under PR granskningsprocessen för att ange hur de tog upp granskarens feedback och förslag. PR-författare kan välja en kommentarstatus från statusrutan.

• Aktiv: standardstatus för nya kommentarer.

• Väntar: problemet i den här kommentaren är under granskning och väntar på något annat.

• Löst: problemet i den här kommentaren åtgärdas.

• Kommer inte att åtgärda: problemet i den här kommentaren noteras men kommer inte att åtgärdas.

• Stängd: diskussionen i den här kommentaren är stängd.

PR-författare och granskare kan spåra PR-förloppet genom att filtrera efter kommentarsstatus, enligt beskrivningen i nästa avsnitt.

Filtrera kommentarer

Du kan välja vilka kommentarer eller uppdateringar som visas på fliken Översikt genom att välja ett filteralternativ i listrutan för kommentarsfilter. Välj till exempel alternativet Vad är nytt filter för att se nya kommentarer och uppdateringar sedan du senast öppnade PR. Varje filteralternativ visar antalet objekt för kategorin.

Redigera filer

För enkelhetens skull kan PR-författare redigera filer direkt i Azure Repos. Du kan till exempel ha accepterat en granskarens föreslagna kodändring och vill göra liknande ändringar någon annanstans.

1. Välj Förrådsfiler, välj PR-källgrenen, välj en fil och sedan Redigera.

   

2. Gör dina ändringar i redigeraren och välj sedan committa för att öppna committdialogrutan.

   

3. I dialogrutan Checka in redigerar du incheckningsmeddelandet om det behövs och väljer sedan Checka in och push-överför ändringarna till din PR.

   

Spåra granskade filer

PR-granskare kan hålla reda på granskade filer genom att välja Markera som granskat från menyn filalternativ eller genom att markera kryssrutan bredvid en fil eller mapp:

Rösta om PR-ändringar

Webbläsare

PR-granskare kan rösta på en PR genom att välja ett röstalternativ i listrutan för omröstning. Granskarikonen på PR-sidan visar en indikation på deras röst.

Röstningsalternativen är:

• Godkänn: godkänner de föreslagna ändringarna i PR. Det här alternativet är bara en röst och godkänner inte PR.

• Godkänn med förslag: godkänner de föreslagna ändringarna i pr med valfria förslag på förbättringar. Det här alternativet är bara en röst och godkänner inte PR.

• Vänta på författaren: ber författaren att granska granskarkommentarerna. PR-författaren bör låta granskarna veta att granska koden igen när PR-författaren har tagit upp kommentarerna. Om en nödvändig granskare anger det här röstningsalternativet blockeras PR-godkännandet.

• Avvisa: anger att ändringarna inte är acceptabla. När du väljer det här alternativet lägger du till en kommentar som förklarar varför. Om en nödvändig granskare anger det här röstningsalternativet blockeras PR-godkännandet.

• Återställ feedback: rensar din röst. Frånvaron av en omröstning hindrar inte att en pr slutförs.

Azure DevOps CLI

Om du vill rösta om huruvida en PR ska godkännas använder du az repos pr set-vote kommandot .

az repos pr set-vote --id
                     --vote {approve, approve-with-suggestions, reject, reset, wait-for-author}
                     [--detect {false, true}]
                     [--org]
                     [--subscription]

Parametrar

Parameter	Obligatoriskt	Beskrivning
--id	Ja	Pull-begärande-ID
--vote	Ja	Nytt röstvärde för pull-begäran (godkända värden: approve, approve-with-suggestions, reject, reseteller wait-for-author)
--detect	Nej	Identifiera organisationen automatiskt (godkända värden: false, true)
--organization	Ja, om en standardorganisation inte har angetts och git-konfigurationsfilen inte anger organisationen	Url för Azure DevOps-organisationen eller ange en standardorganisation med hjälp av: az devops configure -d organization=<organization URL>
--subscription	Nej	Namn eller ID för en Azure-prenumeration eller ange en standardprenumeration med hjälp av: az account set -s <name or ID>

Exempel

Följande kommando röstar för att godkänna PR #21. Kommandot returnerar en tabell.

az repos pr set-vote --id 21 --vote approve --output table

Name            Email                ID                                    Vote      Required
--------------  ------------------- -------------------------------------  --------  ----------
Jamal Hartnett  jamalh@fabrikam.com  00000000-0000-0000-0000-000000000000  Approved  False

Följande kommando tar bort din röst. Kommandot returnerar en tabell.

az repos pr set-vote --id 21 --vote reset --output table

Name            Email                ID                                    Vote      Required
--------------  ------------------- -------------------------------------  --------  ----------
Jamal Hartnett  jamalh@fabrikam.com  00000000-0000-0000-0000-000000000000            False

Slutför en PR

PR-författare kan välja ett alternativ för PR-lösning:

• Slutför: Slutför PR nu om alla obligatoriska granskare har godkänt den och alla nödvändiga grenprinciper uppfylls.

• Ange automatisk slutförande: Slutför PR automatiskt när alla nödvändiga granskare godkänner den och alla nödvändiga grenprinciper uppfylls.

• Markera som utkast: ange pr till utkaststatus för att ange att den inte är redo för granskning. För ett utkast till PR kan du välja Publicera för att ta bort dess utkaststatus och markera den som redo för granskning.

• Avbryt: stäng PR. För en övergiven PR kan du välja Återaktivera för att återställa den.

Mer information om hur du slutför en PR finns i Slutför pull-begäran.

Nästa steg

Slutför pull-begäran

Relaterade artiklar

• Uppdateringsmeddelanden för pull-begäranden
• Om pull-begäranden och behörigheter