Se ho accesso diretto al commit, c'è qualche ragione per creare una richiesta di pull per una piccola modifica che non richiede feedback?

3

Al mio lavoro, usiamo Git come nostro sistema di controllo della versione. Abbiamo un master branch, al quale ho il diretto accesso. A volte devo fare una soluzione banale, come correggere un refuso nella documentazione, e lo standard non scritto nel nostro ufficio è che ottenere i revisori per quello è una perdita di tempo (per favore non discutere questo punto, cambiare la cultura aziendale non è un battaglia che voglio combattere oggi). Quindi ho due opzioni su come andare a fare il mio cambiamento banale:

Crea una richiesta di pull

  1. Crea un ramo fuori dal master
  2. Apporta la mia piccola modifica
  3. Invia la modifica al ramo in remoto
  4. Vai all'interfaccia utente web, che è l'unico modo per effettuare richieste pull sul nostro sistema
  5. Crea una richiesta di pull con zero revisori *
  6. Unisci

Impegno diretto

  1. Apporta la mia piccola modifica
  2. Impegna direttamente sul master

Il risultato finale per i due è lo stesso; il mio cambiamento lo rende padrone. Ma il primo metodo richiede molto più tempo del secondo. Supponendo che la mia squadra non sia interessata a rivedere il cambiamento, c'è qualche motivo, tecnico o organizzativo, che vorrei fare rigmarole la richiesta di pull?

    
posta Thunderforge 17.01.2017 - 17:40
fonte

1 risposta

3

Una volta ho trascorso un'intera settimana a trovare un bug causato da un check-in come questo. Neanche una soluzione banale, ma un cambiamento banale in cui uno sviluppatore pensava che se (p! = NULL) dovesse essere sostituito con if (! P) perché sembra più bello, e non dovrebbe essere rivisto dal codice perché era banale. (Il bug è diventato visibile solo dopo che un collaboratore ha trascorso sei mesi a riscrivere completamente un sottosistema, provocando un arresto anomalo se si sospende il computer per 40-50 secondi e in nessun altro caso).

Assicurati che le revisioni del codice possano essere eseguite con un sovraccarico minimo. Nel mio flusso di lavoro posso fare un piccolo cambiamento oltre alla revisione del codice in due minuti. Non vale la pena correre rischi. Se ti prende più tempo, cambia il tuo processo.

    
risposta data 17.01.2017 - 18:21
fonte

Leggi altre domande sui tag