Review & Test
1. Ga naar GitHub
Open je browser en ga naar github.com/onesteinbv. Controleer of je bent ingelogd met een GitHub-account, zodat je kan reageren op pull requests.
Ga naar github.com/onesteinbv/addons-curq en lees eventueel even de README om te begrijpen hoe het project is opgebouwd.
2. Kies een pull request
Ga naar het tabblad Pull requests bovenaan de pagina. Hier zie je een lijst van alle openstaande PR's. Kies er één uit die je wilt reviewen.
3. Lees de omschrijving
Open de PR en lees de omschrijving. Hierin staat meestal vermeld wat er is aangepast, waarom dat is gedaan en eventueel hoe je het kunt testen. Een goede omschrijving helpt je om snel te begrijpen wat de bedoeling is en of de wijziging logisch is binnen CURQ.
Als de omschrijving niet duidelijk is, voel je vrij om om verduidelijking te vragen (onderaan de pagina).
4. Test de functionaliteit
Open de Runboat omgeving, en installeer eventueel de module(s) uit de PR. Volg de stappen om de nieuwe of aangepaste functionaliteit te testen, dit is meestal beschreven in de module's README of de omschrijving van de PR. Controleer of alles naar verwachting werkt, of er regressies zijn, en of de wijziging goed integreert met bestaande functionaliteit.
5. Optioneel: Controleer de code
Als je bekend bent met software ontwikkeling, kun je ook de code bekijken. Controleer of de code leesbaar, logisch en volgens de stijlrichtlijnen is geschreven. Let ook op mogelijke performance- of security-issues. Je hoeft geen expert te zijn — ook kleine suggesties kunnen waardevol zijn.
6. Plaats de review
Als je feedback hebt, klik dan op Review changes en kies of je commentaar wilt geven, goedkeuring wilt geven, of wijzigingen verwacht. Schrijf duidelijk en constructief en geef eventueel suggesties voor verbetering. Als alles in orde is, geef je je goedkeuring zodat de PR gemerged kan worden.
