Add/refactoring

[mrkrash]

No description provided.

[Cadienvan]

@AngeloAvv abbiamo spostato la PR e ripristinato il tuo commento non risolto.
@mrkrash ha rivisto la sezione "Refactoring vs Riscrittura", ora ti torna?

[Cadienvan]

Proposto solo un suggerimento, semplificando la frase e integrando con un esempio, dimmi se ti piace!

[Cadienvan]

@mrkrash non so se hai letto il suggerimento di @akelity , che ne pensi?

[mrkrash]

@mrkrash sono d'accordo sul suggerimento di @akelity , che ne pensi?

Ho letto i suggerimenti ma, all'atto pratico, significherebbe duplicare concetti piuttosto che riformulare quanto esposto. Sto cercando di capire come poter fare ciò senza appunto duplicarli, cosa che odio profondamente.

[Cadienvan]

@mrkrash in definitiva il capitolo è tuo, di conseguenza se pensi che il contenuto sia già sufficientemente esplicativo possiamo anche procedere. Il punto non è quello di accettare ogni suggerimento ricevuto, quanto di valutarlo!

@akelity sei d'accordo? Come già accennavo, a tutti gli effetti credo che leggere con attenzione il capitolo sia sufficiente a fornire le risposte necessarie, quindi se Mario non dovesse trovare un'alternativa, personalmente approverei comunque il contenuto.

[nicolaerario]

@mrkrash in definitiva il capitolo è tuo, di conseguenza se pensi che il contenuto sia già sufficientemente esplicativo possiamo anche procedere. Il punto non è quello di accettare ogni suggerimento ricevuto, quanto di valutarlo!

@akelity sei d'accordo? Come già accennavo, a tutti gli effetti credo che leggere con attenzione il capitolo sia sufficiente a fornire le risposte necessarie, quindi se Mario non dovesse trovare un'alternativa, personalmente approverei comunque il contenuto.

In realtà il capitolo è di tutti. Cosa voglio dire: se oggi il capitolo partorito da @mrkrash è corretto, rispecchia la sua volontà ed è accettato dalle review, può essere tranquillamente mergiato.
Domani io ho delle idee da integrare? Bene, nuova PR e si va.
E' lo spirito giusto, no?

[akelity]

@mrkrash in definitiva il capitolo è tuo, di conseguenza se pensi che il contenuto sia già sufficientemente esplicativo possiamo anche procedere. Il punto non è quello di accettare ogni suggerimento ricevuto, quanto di valutarlo!

@akelity sei d'accordo? Come già accennavo, a tutti gli effetti credo che leggere con attenzione il capitolo sia sufficiente a fornire le risposte necessarie, quindi se Mario non dovesse trovare un'alternativa, personalmente approverei comunque il contenuto.

Ma si assolutamente!
Era sono un feedback il mio! 😉

[Cadienvan]

Sì bravo @nicolaerario intendevo proprio quello 🤣

[akelity]

Come detto, aggiungiamolo se poi ci sono modifiche si vedranno in seguito :)

[corradopetrelli]

Ottimo!

[AngeloAvv]

LGTM!! 💣

Valutate se è il caso di portar dentro il mio nitpicking :)

[serenasensini]

Ottimo capitolo, chiaro e ben dettagliato!

[serenasensini]

Forse avrei utilizzato un linguaggio in particolare, visto che si parla di variabili mutabili e non...

[mrkrash]

Ok, valuto se riscrivere gli snippet con un linguaggio specifico. Ho optato per del pseudo-codice pensando che un linguaggio specifico potesse essere una barrirera aggiuntiva alla comprensione dello snippet.

[Cadienvan]

Visto che le 6 approval necessarie erano state raggiunte e la revisione grammaticale e di forma di Serena è stata integrata in toto, arriviamo a ben 7 con l'ultima nota pendente riguardante lo pseudo-codice che può benissimo essere accolta in un secondo momento!

Complimenti @mrkrash , tra pochi minuti il tuo capitolo sarà online!