Få feedback genom pull-förfrågningar

Pull-begäranden stöder granskning och sammanslagning av kod i en enda samarbetsprocess. När en utvecklare lägger till en funktion eller en felkorrigering skapar de en pull-begäran för att påbörja processen med att sammanfoga ändringarna i den överordnade grenen. Andra teammedlemmar får sedan möjlighet att granska och godkänna koden innan den slutförs. Använd pull-begäranden för att granska pågående arbeten och få tidig feedback om ändringar. Men det finns inget åtagande att sammanfoga ändringarna. En ägare kan när som helst överge en pull-förfrågan.

Få koden granskad

Kodgranskningen som görs som en del av en pull-begäran är inte bara för att hitta uppenbara buggar. det är vad tester är till för. En bra kodgranskning fångar upp mindre uppenbara problem som kan leda till kostsamma problem senare.

Kodgranskningar hjälper till att skydda teamet från felaktiga sammanslagningar och brutna byggen som minskar teamets produktivitet. Granskningar upptäcker problem före sammanfogningen och skyddar viktiga grenar från oönskade ändringar.

Kodgranskningar uppmuntrar också till och stärker samarbete och kommunikation mellan utvecklare. Och teamet får en tydlig historik över alla ändringar som gjorts mellan huvudgrenen och funktionsgrenarna.

Dela expertis och sprid problemlösningsstrategier genom att använda ett brett spektrum av granskare i kodgranskningar. Genom att sprida kunskaper och färdigheter blir teamet starkare och mer motståndskraftiga.

Ge utmärkt feedback

Högkvalitativa recensioner börjar med feedback av hög kvalitet. Nycklarna till bra feedback i en pull-begäran är:

• Låt rätt personer granska pull-förfrågan.
• Kontrollera att granskarna vet vad koden gör.
• Ge konstruktiv och konkret feedback.
• Svara på kommentarer i tid.

När du tilldelar granskare till en pull-begäran måste du välja rätt uppsättning granskare. Granskare bör känna till hur koden fungerar, men även inkludera utvecklare som arbetar inom andra områden så att de kan dela med sig av sina idéer.

Ange en tydlig beskrivning av ändringarna och ange en version av koden som har korrigeringen eller funktionen som fungerar i den. Granskare bör göra ett försök att ge feedback om ändringar som de inte håller med om. Identifiera problemet och ge specifika förslag på vad som kan göras på olika sätt. Den här feedbacken har en tydlig avsikt och är enkel för ägaren av pull-begäran att förstå.

Ägaren av pull-begäran bör svara på kommentarer, acceptera förslag eller förklara varför de avböjer att tillämpa dem. Vissa förslag är bra, men kan ligga utanför omfånget för pull-begäran. Ta dessa förslag och skapa nya arbetsobjekt och funktionsgrenar separat från pull-begäran för att göra dessa ändringar.

Skydda grenar med principer

Det finns några kritiska grenar på en lagringsplats som teamen förlitar sig på att alltid vara i god form, till exempel, grenen main. Teams kan kräva pull-begäranden för att göra ändringar på dessa grenar med plattformar som GitHub och Azure DevOps. Utvecklare som push-överför ändringar direkt till de skyddade grenarna får sina push-meddelanden avvisade.

Lägg till ytterligare villkor för att hämta begäranden för att framtvinga en högre nivå av kodkvalitet i nyckelgrenar. En ren version av den sammanfogade koden och godkännande från flera granskare är några extra krav som ofta används för att skydda nyckelgrenar.

Lära sig mer

GitHub har omfattande dokumentation om hur du föreslår ändringar i ditt arbete med pull-begäranden.

Läs mer om att ge bra feedback i kodgranskningar och använda pull-begärandemallar för att ge vägledning till granskarna. Azure DevOps erbjuder också en omfattande upplevelse för pull-begäranden som är enkel att använda och skalar efter behov.