Eerstejaars - kosterm14/portfolio GitHub Wiki
Pull Request Review (24/05)
Ik heb vandaag de eerstejaars studenten geholpen door hun pull requests te reviewen. Ze maakten vandaag voor het eerst PR's en ik legde ze uit hoe je dit doet en reviewde dit hierna. Ik heb twee groepjes geholpen met een PR:
Ik legde ze in mijn review uit hoe je het best PR's kunt maken en wat de inhoud moet zijn. Ook heb ik een voorbeeld laten zien van mijn eigen PR zodat ze wat inspiratie hadden.
Links om ze te helpen
Ik heb sommige studenten die wat minder wisten zelfs nog wat extra hulp gestuurd door ze te linken naar handige artikelen die ze goed kunnen gebruiken. Dit zijn artikelen die wij vorig jaar ook kregen dus ze zijn sowieso bruikbaar.
De eerste link is een artikel over het verbeteren van de user interface van hun websites met behulp van de ui stack. Dit is een link die koop ons vorig jaar stuurde en dit leek me handig om met de eerstejaars te delen omdat ze net een nieuwe sprint gestart zijn over UI.
De tweede link is een link die Joost vorig jaar stuurde over pull requests. In dit artikel wordt besproken waarom de grootte van pull requests belangrijk is, wat de voor- en nadelen van kleinere pull requests zijn, en hoe je ermee aan de slag kunt gaan. Dit is handig omdat ze net beginnen met PR's en hier dus alle info over kunnen gebruiken.
Als derde link stuurde ik ten slot ook het pull request template wat wij nu moeten gebruiken. Dit is een goed voorbeeld voor hoe ze het uiteindelijk moeten gaan doen en dus erg handig voor ze!
Ook zag ik tijdens het reviewen dat er gebruik gemaakt werd van nesting selectoren en gaf toen aan dat dat niet nodig is. Hier stuurde ik ook een artikel voor.