Je eerste pull request reviewen

Wat je gaat bouwen

Aan het einde van deze zelfstudie heb je:

• Een pull request geopend in de PR-workbench van Control Center
• De diff gelezen en een inline reactie achtergelaten
• Een reviewverdict ingestuurd
• De PR gemerged

Vereisten: Je eerste agent starten is afgerond en er is één open pull request op een repo die in deze werkruimte is geregistreerd. Kies voor de reviewstappen een PR die je niet zelf hebt geopend — de knop Review is verborgen op je eigen PR’s. Mergen werkt nog wel op een PR die je zelf hebt gemaakt.

Stap 1: De pull request openen

1. Klik op Pull requests in de zijbalk
2. Zoek je PR en klik erop

Controlepunt. De PR opent als een workbench met tabbladen — Overzicht, Diff, Broncodebeheer, Chat, Acties — met focus op Overzicht: de titel en acties, de beschrijving en de activiteitentijdlijn en een zijbalk met status, reviewers, toegewezen personen, controles en gewijzigde bestanden. Als je daar op een bestand klikt, spring je ernaartoe in het tabblad Diff. Het tabblad PR-review is pas geopend zodra iemand om een AI-review vraagt.

Lege wachtrij met een “connect GitHub”-prompt

Die toestand betekent dat de server geen GitHub-token heeft — je client heeft er nooit een. Stel er een in onder Instellingen → Werkruimte → Profiel en identiteit → Codehosting en ververs daarna de lijst.

Stap 2: De diff lezen

Open het tabblad Diff. Links staat een inklapbare bestandsstructuur en de strook boven de diff bevat een selector voor het commitbereik — beperk de diff tot één commit of een reeks ervan — plus het aantal bestanden en de voortschrijdende toevoegingen en verwijderingen.

Met de instellingenknop in de diff-werkbalk bepaal je hoe de diff wordt weergegeven:

• Gesplitst of samengevoegd weergeven
• Regelterugloop of lange regels scrollen
• Ligaturen van het codelettertype

Markeer een bestand als bekeken om je voortgang in een grote diff bij te houden. Toetsenbordnavigatie staat in Sneltoetsen.

Stap 3: Een inline reactie achterlaten

1. Beweeg de muis over een regel in de diff — in de kantlijn verschijnt een kleine +-knop
2. Klik erop om een reactie op die regel te plaatsen, of sleep hem eerst verticaal om een bereik te selecteren
3. Typ je feedback. Het invoerveld biedt twee manieren om te versturen:
   • Review starten (of Aan review toevoegen als er al een review loopt) zet de reactie in de wachtrij. De thread toont Review in wachtrij totdat je in stap 4 een verdict instuurt.
   • Losse reactie toevoegen plaatst de reactie meteen op GitHub. De thread toont Publiceren… en daarna Gesynchroniseerd, en biedt een nieuwe poging als het plaatsen mislukt.

Gebruik in deze zelfstudie Review starten, zodat de reactie met het verdict meegaat.

Een reactie in de wachtrij staat nog niet op GitHub

Review starten / Aan review toevoegen houden de reactie lokaal vast totdat je een verdict instuurt. Alleen Losse reactie toevoegen plaatst meteen. De tekst van de review die je in stap 4 schrijft, is een apart concept.

Stap 4: Een review versturen

1. Klik op Review in de actierij van Overzicht — dezelfde knop staat ook in de werkbalk van het tabblad Diff. Hij is verborgen op PR’s die je zelf hebt gemaakt en op een gesloten of gemergede PR.
2. Schrijf een samenvatting in het invoerveld. Die wordt automatisch als concept opgeslagen, zodat je de overlay kunt sluiten en er later op terug kunt komen.
3. Kies een verdict:
   • Wijzigingen aanvragen
   • Reageren
   • Goedkeuren

Het verdict is de knop zelf. Na Goedkeuren of Wijzigingen aanvragen krijgt de knop het label Goedgekeurd of Wijzigingen aangevraagd. Bij een Reageren-verdict blijft het label Review.

Controlepunt. De knop toont “Goedgekeurd” of “Wijzigingen aangevraagd” als je een van die twee hebt gekozen.

Stap 5: Mergen

De knop Merge is aanwezig zodra GitHub de PR als mergebaar rapporteert — hij is niet afhankelijk van je eigen goedkeuring. De kleur geeft gereedheid aan, niet toestemming: primary wanneer de vereiste goedkeuringen en controles groen zijn, secondary terwijl een controle nog loopt, destructive wanneer een gevraagde reviewer niet heeft goedgekeurd of een controle mislukt.

1. Klik op Merge
2. Kies een methode: Squash and merge, Create a merge commit of Rebase and merge
3. Bewerk bij de eerste twee de vooraf ingevulde commit-titel en het commit-bericht
4. Bevestig

Eventuele openstaande problemen — “Controles mislukt”, “Some reviews are pending” — worden in het uitklapvenster vermeld voordat je bevestigt.

Mergen kun je niet terugdraaien

Mergen pusht commits naar GitHub. Het komt nooit in de undo-stack van Control Center terecht en kan van hieruit niet ongedaan worden gemaakt.

Na het mergen wordt een domeingebeurtenis PrMerged afgevuurd; analytics en meldingen verbruiken die. Zie Domeingebeurtenissen voor wat er nog meer naar luistert.

Reviewen met AI

Control Center kan een reviewerpipeline op een open PR zetten, maar alleen als je erom vraagt: het sjabloon pr_review heeft een handmatige trigger en verder niets.

1. Open het overloopmenu in de actierij van Overzicht en kies Ask AI. Control Center start de run en opent het tabblad PR-review, zodat je de specialisten aan het werk kunt zien.
2. Reviewer-agenten leggen hun bevindingen vast als reviewnodes in de ruimte van de PR; een laatste stap bundelt ze tot één rapport.

De pipeline publiceert bewust niets naar GitHub — hij eindigt na het bundelen. Publiceren is een aparte stap die je zelf zet, met de knop Publiceren naar GitHub op het tabblad PR-review. Zie AI-gestuurde review gebruiken.

Samenvatting

Je hebt een PR geopend in de workbench, de diff met syntaxmarkering gelezen, een inline reactie achtergelaten, een verdict ingestuurd en gemerged — zonder de app te verlaten.

Volgende: Je eerste pipeline bouwen om workflows met meerdere stappen te automatiseren.