Template pull request. DO NOT MERGE.

[snellejelle99]

Geef je pull request een label en een naam die bij de issue past.
voorbeeld naam:
issue: "Button staat scheef op pagina 1 #4 "
Pull request naam: "Fixed #4 Scheve button op pagina 1 recht gezet"
met /# kun je references naar andere issues en pull request toevoegen gebruik dit in je omschrijvingen en titels

Koppel je pull request na het maken aan een issue! Dit doe je in het scherm van de issue.

Omschrijving

Zet hier kort neer wat en hoe je iets gefixed hebt.
bijvoorbeeld: "coordinaten van button aangepast."

Risico's

Zet hier de eventuele risico's van deze aanpassing neer copy paste deze uit
Voorbeeld: 'Hoog risico, aanpassing kan er voor zorgen dat programma niet meer te starten is.'
Of: 'Laag risico, UI aanpassing.'

---

Tests

Zet hier neer hoe de feature getest moet worden. Kopieer deze uit de issue of maak ze hier. Een pull request moet tests hebben! Het is de bedoeling dat minimaal 1 van de reviewer de tests ook uitvoert.

Voorbeelden:

Test 1 "Applicatie afsluiten" :

Voorbereiding:

• Start het programma op.

Test

• Klik op het kruisje rechtsboven in het scherm.
• Controleer of het programma is afgesloten,

Geslaagd als :
Het programma is afgesloten.

---

Test 2 "Applicatie minimaliseren" :

Voorbereiding:

• Start het programma op.

Test

• Klik op het minnetje rechtsboven in het scherm.
• Controleer of het programma is geminimaliseerd,

Geslaagd als :
Het programma is geminimaliseerd.

---

[snellejelle99]

Checklist voor de reviewers

• Naam en omschrijving correct? (pull requests zouden door moeten kunnen gaan voor release notes)
• Risico's correct ingeschat?
• Voldoende Test cases present om de wijziging te coveren?
• Code quality goed? (Geen gekke constructies, correcte naamgeving, geen redundant code, leesbaarheid, comments, etc?)
• Unit tests indien mogelijk?
• Tests cases uitgevoerd?

Iets nettere versie:

• Naam en omschrijving.
• Risico's
• Test cases
• Code quality
• Unit tests
• Test cases geslaagd

Indien niet alle test cases slagen plaats dan een comment met feedback:
bijvoorbeeld:
"Test 1 programma afsluiten:
Ik druk op kruisje maar het programma sluit niet af"

Indien de branche niet build of je een exception tegenkomt tijdens het testen. Plaats de errorlog dan in een comment met aanvullende feedback.