Quando si effettua una correzione a un commit precedente, dovrei eseguire il rebase o aggiungere un commit di correzione separato?

11

Uno scenario comune nello sviluppo del software è la revisione del codice del codice di qualcun altro. Uno strumento comune per farlo è aprire una richiesta di pull.

La mia domanda è, quando i problemi vengono trovati nella revisione, dovrebbero le modifiche

  1. essere impegnati separatamente (nuovo commit)
  2. o dovrebbe essere modificato il commit esistente (presupponendo che nessuno stia ramificando dal commit precedente ... poiché riscrivere la cronologia da un ramo condiviso è una cattiva notizia).

Per il primo scenario, è facile tenere traccia delle modifiche incrementali, sebbene aggiunga un po 'di rumore alla cronologia del commit. La seconda opzione ha i pro ei contro opposti.

    
posta loeschg 03.04.2015 - 18:23
fonte

1 risposta

23

Si presume che la correzione non introduca nuovi problemi e corregge quelli vecchi completi. Ma molte correzioni meritano una revisione autonoma, e questo è probabilmente molto più semplice quando le modifiche incrementali possono essere esaminate separatamente.

    
risposta data 03.04.2015 - 18:34
fonte

Leggi altre domande sui tag