Nella nostra esperienza il “pair programming” da remoto è compito arduo, gli strumenti se si trovano sono estremamente acerbi, e la semplice condivisione dello schermo non ha neanche lontanamente lo stesso effetto dell’essere seduti uno di fianco all’altro. Se in più ci mettete l’ovvia magagna delle time zone il disastro è servito.
Il nostro strumento prediletto sono quindi diventate le code review.
- Se fai push su master o develop direttamente ti tagliamo “i ditini”: apri una pull request, per tutto.
- Se la pull request è troppo grossa qualcuno molto rapidamente te la falcerà.
- Se la pull request contiene commit che non ci azzeccano uno con l’altro, verrai cazziato e qualcuno molto rapidamente te la falcerà.
- Se fai merge delle tue pull request ti tagliamo “i ditini”: qualcun altro farà review e approverà la PR.
Chi fa review sviluppa uno skill detto “gran scassa…”, soprattutto quando subite le prima review. Poi:
- I coding standard vengono pian piano assorbiti da tutti
- Lo stile di scrittura del codice si uniforma
- La dimensione delle PR è sempre piccola e la PR sono ben strutturate in modo da semplificare al massimo la review
- La conoscenza inevitabilmente si diffonde
Alla fine vi ritrovate ad apprezzare l’immenso favore che vi ha fatto chi vi ha fatto le prime review. Oppure diventiamo tutti dei “gran scassa…” e non ce ne accorgiamo più ;-)