Closed ceolonimatteo closed 6 years ago
@ceolonimatteo questa è una versione preliminare, non pronta per una PR. Prima di mandare una PR devi almeno testare il codice localmente sulla tua macchina e vedere che funzioni. Questa versione non è nemmeno lontanamente pronta per un possibile merge e c'è ancora molto lavoro da fare. Chiedi a @Styddy o @simonecasile per dei consigli su quali blocchi usare. Prendi spunto da: https://github.com/Open-Education-Polito/oep-python-interactive-book/blob/master/capitoli/Settimo_Passo_Funzioni.rst per avere un'idea di blocchi e che senso dare al codice. Testa e guarda il risultato compilato prima di fare una PR. Grazie
@ceolonimatteo dopo aver fatto le modifiche riguarda tutto il codice compilato perchè alcuni errori sono proprio visibili nell'html.
@ceolonimatteo leggi bene ogni singolo messaggio di review
@ceolonimatteo bene, siamo pronti per il merge. Ci sono un po' di problemi nella tua git history, dobbiamo sistemare prima di fare il merge.
@ceolonimatteo merged! 👍
Descrizione
Eseguito capitolo 5, da fare la prima review
Cambiamenti
Review