Diversi team della mia azienda praticano un flusso di lavoro di revisione del codice che non avevo mai visto prima. Sto cercando di capire il pensiero che sta dietro, con l'idea che c'è valore nel rendere coerente l'intera azienda. (Contribuisco a più codebase e sono stato innescato dalle differenze in passato.)
- L'autore del codice invia una richiesta pull
- Il revisore esamina il codice
- Se il revisore approva, lascia un commento sulla falsariga di "Sembra buono, sentiti libero di unire"
- Se il revisore ha dubbi, lascia un commento del tipo "Correggi i problemi minori X e Y, quindi unisci" (Per modifiche importanti, torna al passaggio 2)
- L'autore del codice apporta modifiche se necessario, quindi unisce la propria richiesta pull
Ho le seguenti preoccupazioni:
In caso di approvazione al passaggio 3, questo flusso di lavoro crea un roundtrip apparentemente non necessario per l'autore della richiesta pull. Il revisore, che sta già esaminando il codice, potrebbe semplicemente fonderlo immediatamente.
Nel caso in cui vengano richieste modifiche al passaggio 3, l'agenzia per unire la richiesta pull ora spetta esclusivamente all'autore del PR. Nessuno oltre all'autore esaminerà le modifiche prima della fusione.
Quali sono alcuni altri vantaggi o svantaggi di questo flusso di lavoro? Questo flusso di lavoro è comune ad altri team di progettazione?