È corretto chiedere ai contributori di rebase delle loro richieste di pull su github

25

Mantengo un repository github relativamente popolare.

Quando una richiesta pull è valida per unire, di solito chiedo all'autore di rebase a un singolo commit prima di unirlo (specialmente quando ci sono state più piccole modifiche).

Questa è una buona pratica del git? Questa etichetta GitHub è accettabile / standard?

Quindi alcuni vantaggi:

  • Ottengo una bella cronologia di commit pulita nei registri di commit
  • Non ho bisogno di modificare il commit da solo
  • Delega parte del lavoro

Alcuni possibili inconvenienti:

  • Non sono sicuro che questa sia una buona etichetta
  • Non sono sicuro che questa sia una buona pratica del git
  • Di solito ho già chiesto alcune altre modifiche - questa è un'altra e non voglio scoraggiare i contributori.
posta Pablojim 26.03.2014 - 18:46
fonte

1 risposta

16

Per quanto riguarda Git, è una sorta di guerra santa se sia sufficiente unire rami o rebase commit sull'ultima versione del ramo in cui ci si sta fondendo. Ci sono molte conversazioni su quale sia meglio se fai una ricerca rapida su Programmers.SE .

Per quanto riguarda l'etichetta che sta dietro, lascia che sia affrontato da una prospettiva pratica. Quando si ha a che fare con un nuovo codice proveniente da qualcun altro, è sempre meglio convincerli a unire le ultime modifiche dal ramo oa rebase di recente prima di unire per garantire un'unione pulita. Ricorda, hanno scritto il codice in modo che di solito siano di gran lunga i più qualificati per affrontare eventuali conflitti di unione / rebase. Personalmente non vedo un problema con esso, e vedo sempre questa richiesta da altre persone. Per me, se non ci sono conflitti, spesso lo faccio da solo perché è un aggiornamento di due secondi che git può applicare da solo. Ma se ci sono conflitti allora chiederò sempre all'autore originale del codice di occuparsene da solo.

Inoltre, per GitHub (come minimo) in particolare, visualizzeranno un link al tuo file CONTRIBUTING al di sopra di qualsiasi tentativo di pubbliche relazioni in modo che sia un buon posto per delineare le tue aspettative e molti progetti includono che si uniranno solo fino ad oggi filiali.

    
risposta data 02.05.2014 - 02:44
fonte

Leggi altre domande sui tag