Come gestire la completa riscrittura di un file per quanto riguarda la revisione del codice

0

La mia situazione è questa: sto contribuendo a un progetto Symfony / PHP. Ho dovuto reskin completamente una pagina nell'applicazione. La logica del controller è quasi totalmente invariata . Ho solo dovuto cambiare il file modello (denominato index.html.twig). Ma i cambiamenti sono completamente diversi. Se faccio una richiesta di pull con queste modifiche contro il ramo RC, il diff per questa richiesta pull sarà dappertutto. Per qualche motivo questo mi sembra sbagliato. Poiché questo file modello è una "nuova" modifica, attualmente sto risolvendo questo problema creando un nuovo file denominato "index_new.html.twig" e modificando la logica del controller per fare riferimento a questo file. In questo modo quando faccio una richiesta pull, il file modello verrà visualizzato come un nuovo file e sarà ovvio che il controller faccia riferimento al nuovo file. Ma anche questo sembra sbagliato perché sembra essere una soluzione hacky. Inoltre, cosa succede se questa pagina è ridisegnata di nuovo? Potrei dover fare "index_new_new.html.twig".

Qual è il modo ottimale per gestire questa situazione?

    
posta Kodos Johnson 08.11.2018 - 20:09
fonte

1 risposta

1

Dai un'occhiata al codice di base per vedere quale è il layout preferito. O anche chiedere ai revisori se puoi sul loro forum / email / im. In questo modo il tuo contributo è coerente.

Per quanto riguarda le dimensioni. Supera i test? Ed il cambiamento è coeso?

Se hai rotto i test, probabilmente il tuo cambio non sarà accettato in entrambi i casi. Se correggi i test, assicurati che tali modifiche siano facili da capire. Potrebbero persino guidarti a capire se il cambiamento è coeso o meno.

Se il cambiamento non è coesivo, trova le modifiche coesive più piccole che lo compongono. Rompi il tuo impegno e offri ognuno come un tiro separato.

Alla fine della giornata, non si tratta di differenze piuttosto, ma di comunicazione e comprensione chiare. Se i diff sono disordinati, ma i test sono chiari e il tuo commento di commit è diretto, è meglio di un cambiamento di una riga, test criptici e un commento buttato via.

    
risposta data 09.11.2018 - 09:39
fonte

Leggi altre domande sui tag