Etichetta per la modifica della richiesta di pull di qualcuno

18

Possiedo un repository su GitHub a cui qualcuno ha inviato una richiesta pull con un singolo commit. Voglio solo implementare la sua soluzione parzialmente, e utilizzare circa la metà del codice cambia l'utente effettuato. Cosa dovrei fare in questa situazione?

Crea un ramo della sua versione, quindi torna indietro e copia e incolla il "vecchio" codice che voglio preservare dalla versione originale in un secondo commit. Questo potrebbe rendere il diff tra i commit sembra più grande di quanto non sia in realtà e butta via cose come git blame .

Copia e incolla il codice che voglio mantenere dal suo commit in un nuovo commit diverso . Ciò significa che non riceve credito per il suo prezioso contributo al codice.

Come sopra, copia un po 'del suo codice in un nuovo commit, ma cambia l'autore del commit a lui invece di me . Tecnicamente non ha scritto il codice esatto che è stato commesso, quindi non sono sicuro se questo è disapprovato. Ma almeno ottiene l'attribuzione per quelle linee che vengono utilizzate.

    
posta IQAndreas 10.10.2013 - 15:24
fonte

1 risposta

35

Vado con l'opzione 4: spiega al contributore perché la sua richiesta di pull non si adatta agli obiettivi del progetto (e nel processo dà al contributore la possibilità di spiegare perché pensa che fa ) e chiedigli di inviare nuovamente una nuova versione contenente solo le modifiche che si adattano al progetto.

Questo ha tre vantaggi:

  1. Non devi fare il lavoro; -)
  2. Il contributore avrà una migliore comprensione dei tuoi obiettivi, rendendo più probabile che i contributi futuri non abbiano bisogno di modifiche
  3. Oltre a mantenere un senso di proprietà, il contributore è riconosciuto e riconosciuto come un contributo utile. Tutto ciò li rende più propensi a continuare a contribuire
risposta data 10.10.2013 - 15:30
fonte

Leggi altre domande sui tag