recensione del codice e poi? [duplicare]

0

Nel mio team abbiamo una politica che ogni codice dovrebbe essere esaminato da almeno un altro sviluppatore. Una volta eseguita la revisione, il revisore fa semplicemente clic sul pulsante di fusione in modo che il codice finisca nello sviluppo. Di recente è emersa un'idea secondo cui il revisore non dovrebbe essere in grado di unire direttamente, ma dovrebbe passare la revisione del codice a un membro degli sviluppatori con il permesso di fondersi effettivamente (2-3 persone). La dimensione della squadra è di circa 8 persone che cercano di lavorare secondo Scrum. Vorrei ricevere un feedback sul fatto che questa sia una buona idea e in tal caso in quali situazioni.

    
posta Tomasz Grobelny 09.09.2016 - 16:28
fonte

1 risposta

3

L'idea di una revisione del codice richiesta prima che finisca in un ramo condiviso è una buona idea. Naturalmente, le persone che stanno facendo la revisione del codice dovrebbero essere le persone con la conoscenza del sistema in fase di sviluppo (compresi i requisiti, l'architettura e la direzione futura) piuttosto che i membri di qualsiasi team. Non ha molto senso che una nuova marca venga assegnata per fare revisioni del codice ed essere in grado di unirla, a meno che non comprendano il contesto più ampio.

L'idea di passare una revisione del codice non ha comunque molto senso per me. Se ci sono un piccolo numero di sviluppatori senior, allora quelle persone dovrebbero essere quelle che fanno le revisioni del codice e la fusione, insieme al tutoraggio e al coaching degli sviluppatori meno esperti per aiutarli a raggiungere un livello più alto di conoscenza. Man mano che più persone acquisiscono esperienza e conoscenza, a più persone può essere data la responsabilità di condurre revisioni del codice e unire il software.

Non penso che Scrum ti offrirà alcuna guida, ma lo Lean Software Development lo farà. Sia il cambio di contesto che il movimento sono sprechi che, in un ambiente snello, dovrebbero essere ridotti o eliminati. Nel tuo stato attuale, hai una persona che ha bisogno di cambiare contesto e lavorare. Nello stato futuro proposto, hai due persone che devono cambiare contesto e passare messaggi tra loro.

    
risposta data 09.09.2016 - 16:40
fonte

Leggi altre domande sui tag