Een pull requests heeft een Developer en een Reviewer nodig. Idealiter dus een 4-ogen principe. Hierbij onze cloud best practices op een rijtje!
Ja, het is zover. Je hebt keihard gewerkt om alle features waar je team zich aan het begin van de sprint aan gecommitteerd heeft af te krijgen. Met je team. Dus dat betekent automatisch ook dat iemand anders dan jijzelf de review doet en jouw code of wijziging checkt. Wat zijn onze cloud best practices voor het reviewen van pull requests? We nemen je mee door de voor sommigen nog ‘hidden feature’ in Azure DevOps en geven tips voor zowel de Developer als de Reviewer en we sluiten af met onze gouden tip.
Maak het jezelf en je team makkelijk door op een eenduidige manier te werken. Gebruik tooling in je eigen voordeel en maak afspraken over de kwaliteitseisen die je stelt. Niet alleen aan je code, maar ook aan je werkproces.
Geloof ons. Het maakt je leven als Developer een stuk makkelijk. Je hebt overzicht, waarborgt de snelheid en kwaliteit. En je maakt het een gezamenlijk resultaat.
Ben je al bekend met het 4-ogen principe? We ‘hameren’ er hem gewoon maar weer in hoor. Joost-Jan schreef er eerder al over en ook Anton bespreekt het in de Golden Path videoreeks. Moraal van het verhaal? Samenwerking is key! Lees meer op TeamValue - The Golden Path.
Conclusie: een Developer maakt de code om de functionaliteit zoals die beschreven is in de user story te realiseren en maakt een pull request. Een mede-Developer voert een peer review uit waardoor het 4-ogen principe wordt gewaarborgd. Laatstgenoemde keurt het pull request na het afronden van de review en het verwerken van het eventuele review commentaar goed.
En dan de gouden tip! Het (peer) review proces in een team is super belangrijk! Maak daarom de afspraak in je team dat wanneer de build breekt en/of een bug die tijdens een review gezien had moeten worden, de reviewer hiervoor verantwoordelijk is. Wordt er iets gemist? Dan moet de reviewer trakteren! Let maar eens op hoe serieus de review taak na het maken van deze afspraak genomen wordt.. ;-) Bijkomend voordeel als het dan wel een keertje misgaat: iedereen leert ervan en er is bijvoorbeeld lekker taart op kantoor. Liever niet al te dik worden van teveel traktaties? Volg dan bovenstaande tips. Moet jij eens opletten hoe snel jullie team de waarde inziet van jullie eigen gouden pad.
Meer lezen van onze cloud collega’s? > TeamValue - Blog
TeamValue is een jong en snelgroeiend bedrijf met een stevige basis. Wij zijn op zoek naar mensen die IT-skills hebben en het ook tof vinden om bijvoorbeeld workshops te geven. Samen op zoek naar de beste oplossing voor een klant. Daar gaat het ons om!
Een hele dynamische rol, waarin meer dan alleen technische skills gevraagd worden! Jij bent die Fullstack Software Developer die weet hoe je bedrijfskritische applicaties bouwt. Al jouw oplossingen moeten 100% geautomatiseerd uitgerold worden.
Jij bent die Cloud Engineer die de oude wereld snapt, daar veel ervaring in hebt maar eigenlijk stiekem niks liever wil dan de nieuwe wereld van Modern Service Management en de public cloud induiken. Bezit jij scripting kwaliteiten, sta je voor proactief beheer en ga je ‘aan’ op het oplossen van (on)mogelijke problemen? Het team kan niet wachten je te ontmoeten.
Jij hebt lef! Want je durft kritisch te zijn tegen onze klanten. Je helpt de klant écht een stap verder. Je denkt mee, stelt vragen en geeft een weerwoord. Want TeamValue gaat voor het beste resultaat.