Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Tijdens sprint sprint 3 had Thijmen een pull request aangemaakt voor de functionaliteit gebruikers lijst ophalen. Ik ben deze pull request gaan nakijken om te bepalen of hij van voldoende kwaliteit was. Omdat er een paar stukjes code in stonden die terwijl hij aan het programeren was veranderd waren moesten die delen ook aangepast worden in zijn pull request om ervoor te zorgen dat er geen niet werkende code op de develop kwam. Ook werd er in een van de functies het wachtwoord mee opgehaald, deze was voor deze functie niet van belang dus ook daar liet ik een comment over achter. Thijmen zag dat ik de comments had achter gelaten en was aan de slag gegaan met het aanpassen, toen dit gedaan was keek ik er weer overheen en vond ik een verandering die bestaande functionaliteit in de knoop zou kunnen brengen dus ook hier liet ik een comment over achter. Hier kreeg ik vervolgens een reactie op die mij gerust stelde en vervolgens kon het dus gemerged worden. Nu hadden we dus een voldoende stuk code die goede functionaliteit toevoegde en niks anders in de problemen bracht.


Tijdens het uitvoeren van mijn rol heb ik geleerd om soms iets zorgvuldiger te kijken na het werk dat ik na kijk en ook niet te vergeten de code ook zelf te runnen om te kijken of het bij mij ook goed gaat, dit deed ik soms wel soms niet. Voor een volgende keer kan ik hier dus beter op letten.

Deze rol past best goed bij mij vind ik zelf, ik vind het vaak wel leuk om werk na te kijken en ben er ook wel redelijk zorgvuldig in. Ook ben ik zelf soms wel een beetje een perfectionist als het gaat om dingen gelijk en consistent krijgen dus dit kan soms ook hiermee helpen.Tijdens 

6. Competenties

7. Leerdoelen

...