Git Pull Request

Met een Pull Request vraag je aan je samenwerkers (in bij DEF: aan je hoofdTAs) toestemming om de wijzigingen in jouw branch toe te voegen aan de branch waar de pull request naar toe gaat, in ons geval de main branch. Om nu jullie opdracht na te laten kijken en terug te mergen met jullie main branch gaan we een pull request (PR) doen. De pull request doe je als je klaar bent met de opdracht. GitHub kijkt dan of alles klopt en runt checks die wij hebben toegevoegd.

Nakijken

Om na te kijken gebruiken wij het review-systeem van Git in de PRs. Als de PR is aangemaakt kan je rechts bovenin een reviewer aanklikken en zet je de link van de PR in de bijbehorende assignment op Brightspace. De reviewer moet jullie TA worden, deze gaat dan nakijken en laat wat comments achter en vraagt misschien om verandering als we vinden dat het nog niet helemaal correct is. Als dit zo is, dan moeten jullie aanpassingen gaan maken, dit kan gewoon weer in dezelfde branch met nieuwe commits (vb: commentaar verwerken). Zodra je alles hebt verbeterd, vraag je opnieuw de review aan van je TA.

Op Brightspace houden we uiteindelijk bij of de opdracht namelijk succesvol is afgerond

Voorbeeld

Nu alles is gesynct op mijn GitHub account ga ik een pull request doen van mijn opdracht branch naar main.

1. Ik klik op ‘Compare & pull request’:

2. Nu zijn we in het pull request menu:

3. We moeten zorgen dat we niet een PR openen naar de DEF-D opdrachten repo, maar naar jouw eigen main branch, dit doe je hier:

4. Bij reviewers moet je jouw nakijker toevoegen, in mijn geval is dit hoofdTA: MartijnSonneveld.

   • Je hoort op de dag zelf wie jouw nakijker is.

5. Zet nu de link van jouw PR op de juiste plek in Brightspace.

   • Voor DEF-D moet je meestal zodra je je pull request hebt aangemaakt dit op Brightspace melden in een assignment. Dat is belangrijk om voor je DEF-D onderwijs je punten te krijgen. Dat doe je door de URL/link te kopieren. Dus de link van de pagina van je PR to kopiëren en die te plakken in de juiste assignment op Brightspace.

6. Martijn heeft nagekeken en was niet tevreden:

7. Ik maak veranderingen en commit en push die, daarna kan ik nog een review aanvragen van Martijn:

8. Deze keer vond Martijn het goed!

Nu willen we mergen met de main branch, squash & merge om precies te zijn!

Nu het is goedgekeurd door de TA en zal hij zorgen dat alles op jullie main branch komt! Dit doet hij door deze stappen te volgen:

• 

• 

• Dan deleten we de branch:

Automatische Checks

Wij hebben een paar automatische checks toegevoegd, de TA’s kijken pas na als al die checks zijn goed gegaan. Vraag dus ook pas voor een review als de checks goed zijn gekeurd. Als de checks niet goed zijn, lees goed wat er mis is, pas dat aan in je opdracht, commit & push opnieuw, dan gaan de checks opnieuw kijken of het nu wel in orde is.

Deel je werk met je groepsgenoot

Als je je werk wilt delen met je groepsgenoten, zodat hun repo ook geüpdate is met jullie opdracht, kan dit ook via een PR. Maak dus nog een PR aan vanuit jouw branch waar je in hebt gewerkt, naar de main branch van de forks van je groepsgenoten.

1. Maak dus nog een PR aan en kies hier je groepsgenoten:

2. Mijn groepsgenoot is Danielle, ik push mijn opdracht branch naar haar main branch:

3. Zij moet dan ook de reviewer zijn, samen met de hoofdTA, en de PR pas accepteren als de TA de normale PR heeft goedgekeurd.