Come riconciliare "misc fix" si commette con le richieste pull?

4

Lavoro in una piccola azienda, dove ero l'unico sviluppatore. Di recente abbiamo dato il benvenuto a poche altre persone nel team e abbiamo iniziato a pensare a come organizzare gli sforzi di sviluppo.

Sono praticamente un principiante in termini di gestione del codice, in quanto questo è il mio primo lavoro fuori dal college. Come parte di un test, ho provato a lavorare utilizzando richieste di pull la scorsa settimana (usiamo GitLab) . Ha reso molto più semplice la discussione delle modifiche al codice relative a una singola funzione e mi piacerebbe continuare a utilizzarle.

Tuttavia , abbiamo scoperto che c'erano un sacco di piccoli miglioramenti che da soli non sembravano meritevoli di una richiesta di pull. Errori di battitura, piccoli commenti, correzioni di bug molto piccole, a volte cose completamente indipendenti che abbiamo notato mentre stavamo implementando le grandi funzionalità.

In che modo vengono gestiti generalmente questi casi?

  • Dovremmo aggiungere "fix" direttamente sul nostro dev branch?
  • Dovremmo creare e unire un PR con una sola piccola correzione?
  • Dovremmo raggruppare un gruppo di questi commit in un unico "fix per errori multipli"?
  • (Aggiunto) Dovremmo consentire piccoli commit all'inizio di ogni PR?
  • Qualche altra opzione?
posta F.X. 19.01.2015 - 22:13
fonte

1 risposta

1

Mi piace raggruppare la pulizia del codice in un singolo commit e poi andare da Là. Assicurati di applicare il codice di pulizia PR prima di affrontare gli altri - ridurrai il tuo mal di testa.

    
risposta data 19.01.2015 - 22:38
fonte

Leggi altre domande sui tag