Pull requests aanmaken en beoordelen

Pull requests (PR's) vormen de kern van gezamenlijk ontwikkelen op platforms zoals GitHub. Ze bieden een manier om wijzigingen aan een project voor te stellen en te laten beoordelen voordat ze worden geïntegreerd.

Zie een PR als een formeel verzoek om uw nieuwe code samen te voegen met een hoofdbranch, meestal main of develop.

PR's bieden teams en individuele ontwikkelaars verschillende belangrijke voordelen:

• Codebeoordeling: anderen kunnen uw code controleren op fouten, goede werkwijzen en verbeteringen.
• Bespreking: ze bieden een speciale plek om wijzigingen te bespreken.
• Kwaliteitscontrole: zo komt alleen goedgekeurde, geteste code in het hoofdproject terecht.
• Wijzigingen bijhouden: een duidelijke geschiedenis van wie wat en waarom heeft gewijzigd.

De gebruikelijke werkstroom voor een pull request bestaat uit deze stappen:

1. Maak een nieuwe branch voor uw functie of oplossing.
2. Breng wijzigingen aan en commit ze naar uw nieuwe branch.
3. Push uw branch naar GitHub.
4. Open een pull request op GitHub.
5. Beoordelaars geven feedback en keuren de wijzigingen goed.
6. Voeg uw wijzigingen samen met de hoofdbranch.

Voordat u wijzigingen aanbrengt, is het een goede gewoonte om een nieuwe branch te maken. Zo houdt u uw werk gescheiden van het hoofdproject totdat het klaar is.

Gebruik git checkout -b om een nieuwe branch te maken en ernaar over te schakelen:

Breng nu de nodige codewijzigingen aan voor je functie of bugfix. Zodra je klaar bent, stage en commit je deze naar je nieuwe branch, net als bij elke andere Git-commit.

Nadat je je wijzigingen lokaal hebt gecommit, push je je nieuwe branch naar je externe repository op GitHub. Hierdoor worden je branch en de bijbehorende commits online zichtbaar.

De vlag -u stelt de upstream-branch in, zodat toekomstige opdrachten met git push eenvoudiger zijn.

Nadat je branch is gepusht, vraagt GitHub je meestal om rechtstreeks vanaf de pagina van je repository of via het tabblad 'Pullrequests' een pullrequest aan te maken.

• Selecteer je featurebranch als de head (bron).
• Selecteer de branch main (of doelbranch) als de base.
• Voeg een duidelijke titel en beschrijving toe waarin je je wijzigingen uitlegt.

Na het openen komt je PR in de beoordelingsfase. Teamleden (beoordelaars) bekijken je code, stellen vragen of stellen verbeteringen voor.

• Tabblad Gesprek: Discussies en opmerkingen.
• Tabblad Gewijzigde bestanden: Bekijk de exacte codeverschillen (diff).
• Tabblad Commits: Bekijk afzonderlijke commits in de PR.

Beoordelaars kunnen rechtstreeks opmerkingen bij coderegels plaatsen of algemene feedback geven. Daarna kunnen ze:

• Goedkeuren: De code is klaar om te worden samengevoegd.
• Wijzigingen aanvragen: Er is meer werk nodig voordat de code kan worden samengevoegd.
• Opmerking plaatsen: Algemene feedback geven zonder het samenvoegen te blokkeren.

Op basis van de feedback moet je mogelijk meer commits naar je featurebranch maken.

Nadat je PR is goedgekeurd, kan deze worden samengevoegd met de base-branch. GitHub biedt enkele opties voor het samenvoegen:

• Mergecommit: Behoudt alle commits uit de featurebranch.
• Squashen en samenvoegen: Combineert alle commits tot één nieuwe commit.
• Rebasen en samenvoegen: Speelt de commits van de featurebranch opnieuw af boven op de base-branch.

De gekozen methode heeft invloed op de commitgeschiedenis van je project.

Pullrequests zijn essentieel voor moderne samenwerking in teams. Je hebt geleerd hoe je:

• Een nieuwe branch voor je werk maakt.
• Je branch naar GitHub pusht.
• Een pullrequest start.
• Het beoordelings- en goedkeuringsproces begrijpt.
• Je wijzigingen met het hoofdproject samenvoegt.

PR's goed beheersen is belangrijk om effectief bij te dragen aan elk Git-gebaseerd project!