Abbiamo avuto un dibattito sul nostro team su come dovrebbe essere pulito il ramo Master.
L'applicazione è scritta e gestita da due persone, me (uno sviluppatore) e un designer GUI / UX. Il progettista GUI / UX esegue molti prototipi (o esperimenti di tipo "sandbox") per testare o correggere vari problemi di layout in JS & CSS. Questo lavoro preliminare o provvisorio introduce un codice "sporco" come CSS inline, JS sparsi, formattazione scadente, ecc. Vorrebbe controllare direttamente in Master non appena funzionalmente i suoi obiettivi sono stati raggiunti, e io la fermo.
I miei check in Master sono molto puliti e faccio sempre una Diff per assicurarmi che siano definitivi, formattati e modularizzati. Pulisco le modifiche preliminari o provvisorie che dovevo fare per garantire che il Master fosse "ufficiale". Il Master dovrebbe essere sporco?