Ottenere il massimo dalle code reviews
Mattia Tommasone
@raibaz
https://bit.ly/codereviews-ita
Slides!
About me
- Developer
- Developer Advocate
- Open Source Maintainer
- Open Source Contributor
- Open Source Supporter
- (Ex) Allenatore di calcio giovanile
- (Ex) Insegnante
- (A volte) Speaker
Alzate di mano
Mi sarebbe piaciuto andare per
- Legge di Linus
- Condivisione di conoscenza macro
- Condivisione di conoscenza micro
- Rubberducking
- Pratica di comunicazione
- Discussione più precisa
- Empatia
Errore comune #1:
Errore comune #2:
Too many reviewers
Quanti reviewer?
- Google: due, di natura diversa
- GitLab: catena di reviewer
- Non c'è una regola che vada bene per tutti, ma una volta scelta va seguita
Legge di LinusCondivisione di conoscenza macroCondivisione di conoscenza microRubberduckingPratica di comunicazioneDiscussione più precisaEmpatia
Come aiuto i reviewer?
Anticipa le domande
aka, "se è stato difficile da scrivere, rendilo facile da capire"
- Test coverage migliorata
- Richiesta di modifiche impossibile da contraddire
- Contesto storico
DO's:
- Cerca problemi in tutto il codice
- Suggerisci tutti i miglioramenti possibili
- Mantieni standard alti
- Concentrati solo sul codice e solo su questa PR
- Ricordati della barriera linguistica
- Metti da parte del tempo per fare code review
DON'Ts:
- Essere scortese
- Estendere la discussione ad argomenti diversi dal codice
- Rendere la discussione personale
- Lasciare indietro qualcosa per gentilezza
- Passare tutto il tempo facendo code review
ASSUME GOOD INTENT.
“An amateur is defensive. The professional finds learning (and even, occasionally, being shown up) to be enjoyable; they like being challenged and humbled, and engage in education as an ongoing and endless process.”
— Ryan Holiday, Ego Is the Enemy
Bonus track:
-
Sono da solo
-
Sono un consulente esterno al team
-
Sono un freelance che fa progetti one-shot
Domande? Feedback?
(I will assume your good intent)