Feedback ontvangen met pull-aanvragen

Pull-aanvragen ondersteunen het beoordelen en samenvoegen van code in één gezamenlijk proces. Zodra een ontwikkelaar een functie of een foutoplossing toevoegt, maken ze een pull-aanvraag om te beginnen met het samenvoegen van de wijzigingen in de upstream-vertakking. Andere teamleden krijgen vervolgens de kans om de code te beoordelen en goed te keuren voordat deze is voltooid. Gebruik pull-aanvragen om de lopende werkzaamheden te controleren en vroege feedback te krijgen over wijzigingen. Maar er is geen toezegging om de wijzigingen samen te voegen. ** Een eigenaar kan op elk gewenst moment een pull request afbreken.

Code laten beoordelen

De codebeoordeling die wordt uitgevoerd als onderdeel van een pull-aanvraag, is niet alleen om duidelijke bugs te vinden; Dat is waar tests voor zijn. Een goede codebeoordeling onderschept minder voor de hand liggende problemen die later tot kostbare problemen kunnen leiden.

Codebeoordelingen helpen het team te beschermen tegen slechte samenvoegingen en mislukte builds die de productiviteit van het team verzwakken. Reviews vangen problemen op vóór de samenvoeging en beschermen belangrijke vertakkingen tegen ongewenste wijzigingen.

Codebeoordelingen stimuleren en versterken ook samenwerking en communicatie tussen ontwikkelaars. En het team krijgt een duidelijke geschiedenis van alle wijzigingen die zijn aangebracht tussen de hoofdbranch en de functiebranches.

Expertise kruisbestuiven en strategieën voor probleemoplossing verspreiden door gebruik te maken van een breed scala aan beoordelaars bij code reviews. Het verspreiden van vaardigheden en kennis maakt het team sterker en toleranter.

Geweldig feedback geven

Beoordelingen van hoge kwaliteit beginnen met feedback van hoge kwaliteit. De sleutels voor geweldige feedback in een pull-aanvraag zijn:

• Laat de juiste personen de pull request reviewen.
• Zorg ervoor dat revisoren weten wat de code doet.
• Geef bruikbare, constructieve feedback.
• Reageer tijdig op opmerkingen.

Wanneer u revisoren toewijst aan een pull-aanvraag, moet u de juiste set revisoren selecteren. Revisoren moeten weten hoe de code werkt, maar ook ontwikkelaars opnemen die op andere gebieden werken, zodat ze hun ideeën kunnen delen.

Geef een duidelijke beschrijving van de wijzigingen en geef een build op van de code waarvoor de fix of functie erin werkt. Revisoren moeten proberen feedback te geven over wijzigingen waarmee ze het niet eens zijn. Identificeer het probleem en geef specifieke suggesties over wat er anders kan worden gedaan. Deze feedback heeft een duidelijke intentie en is eenvoudig voor de eigenaar van de pull-aanvraag om te begrijpen.

De eigenaar van de pull-aanvraag moet reageren op opmerkingen, suggesties accepteren of uitleggen waarom ze weigeren om ze toe te passen. Sommige suggesties zijn goed, maar vallen mogelijk buiten het bereik van de pull-aanvraag. Neem deze suggesties en maak nieuwe werkitems en functiebranches gescheiden van de pull-aanvraag om deze wijzigingen aan te brengen.

Branches beveiligen met beleidsregels

Er zijn enkele kritieke branches in een repository waarop teams vertrouwen dat deze altijd in goede staat verkeren, zoals de main branch. Teams kunnen pull-aanvragen vereisen om wijzigingen aan te brengen in deze branches met platforms zoals GitHub en Azure DevOps. Ontwikkelaars die wijzigingen rechtstreeks naar de beveiligde vertakkingen pushen, krijgen hun pushes geweigerd.

Voeg aanvullende voorwaarden toe aan pullverzoeken om een hoger niveau van codekwaliteit af te dwingen in belangrijke branches. Een schone build van de samengevoegde code en goedkeuring van meerdere beoordelaars zijn enkele extra vereisten die vaak worden gebruikt om belangrijke branches te beveiligen.

Meer informatie

GitHub bevat uitgebreide documentatie over het voorstellen van wijzigingen in uw werk met pull-aanvragen.

Lees meer over het geven van geweldige feedback in codebeoordelingen en het gebruik van pull-aanvraagsjablonen om uw revisoren richtlijnen te bieden. Azure DevOps biedt ook een uitgebreide pull-aanvraagervaring die eenvoudig te gebruiken is en indien nodig kan worden geschaald.