11. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 associazioneodooitalia@gmail.com odoo-italia.org Q&A Grazie della tua attenzione @SimoRubi
3. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 “Peer code reviews are the single biggest thing you can do to improve your code” — Jeff Attwood Procedura di controllo fatta da revisore . Cos’è una Review
7. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Revisore ● Anche se non sei un esperto, il tuo contributo è importante. ● Imparare ad usare il software Approccio a una review
4. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 “Dato un numero sufficiente di occhi, tutti i bug vengono a galla” — legge di Linus Revisore image by Manu Cornet under Creative Commons license Autore Mentalità critica
8. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Revisore ● Condividere la conoscenza. ● Aumentare la qualità del codice. ● Controllare aspetti che non possono essere automatizzati. Obiettivi di una review
2. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 Proposta di modifica fatta da autore . Cos’è una PR Creazione di una Pull Request in Github Perchè fare una PR ● richiesta di un cliente ● nuove normative ● migliorie
10. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Autore ● Ragionamento obiettivo ● Non prenderla sul personale ● Focus sul codice: le risposte in PR volant, il codice manent. Interazione con il revisore
5. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Autore ● Sindrome dell’impostore: non sono abbastanza bravo per contribuire. ● Aiuto dall’automazione. [loghi] Approccio a una PR L’ autore è il primo revisore .
9. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Revisore Interazione con l’autore Comunicazione chiara: spiegare motivazioni. Suggerire soluzioni lasciando libertà all’autore. Fare riferimento al codice, non all’autore. Considerare il background dell’autore. Più fai review meglio saranno le tue PR.
1. Odoo Days Italia odoo-italia.org Bari, 4 e 5 novembre 2021 Titolo Comunicazione in PR e Review. Idee e indicazioni per una efficace collaborazione tra autore e revisore. Quando Venerdì 05/11/2021 Autore Simone Rubino In altre parole: Sviluppatore Odoo presso Agile Business Group Riferimenti simone.rubino@agilebg.com
6. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 consigli per Autore ● Descrizione chiara: ○ cosa ○ perché ○ riferimenti ● Non più di 300 righe modificate. ● Deve risolvere 1 problema, aggiungere 1 comportamento o altro. Requisiti per una buona PR
12. Comunicazione in PR e Review odoo-italia.org Bari, 4 e 5 novembre 2021 Riferimenti ● https://kscurtis2000.medium.com/what-is-a-code-review-74f080c33013 ● https://medium.com/swlh/4-rules-of-thumb-for-providing-effective-code-review-feedback-bb188864f50d ● https://kscurtis2000.medium.com/how-to-provide-effective-feedback-based-on-code-review-type-ccace 3d60092 ● https://kscurtis2000.medium.com/inclusive-and-productive-collaboration-in-code-reviews-e7d05c228f4d ● https://kscurtis2000.medium.com/imposter-syndrome-how-to-overcome-it-during-code-reviews-3eb23b eae383 ● https://google.github.io/eng-practices/ ● https://smartbear.com/learn/code-review/best-practices-for-peer-code-review/ ● https://trishagee.com/presentations/code_review_best_practice/ ● https://www.odoo-italia.org/documentazione/12.0 ● https://speakerdeck.com/nnja/code-review-skills-for-pythonistas-djangocon-2018