Sto scrivendo un documento del processo di revisione del codice per il nostro team; non abbiamo mai avuto un processo formale, anche se facciamo qualche revisione del codice.
Ho trovato molti articoli che parlano di quanto sia importante la revisione del codice, ma ho una domanda in particolare che non ho trovato la risposta sul web ... è la parte programmatore della revisione del codice? In altre parole, una recensione del codice dovrebbe essere di 2 persone sedute insieme, andando oltre ciò che è nel codice, o dovrebbe essere 1 persona a guardare il codice di un'altra persona?
Molto spesso, i nostri progetti sono svolti da un team di 2 persone, quindi prima che venga inviato al QA, queste 2 persone dovrebbero semplicemente sedersi insieme per esaminare l'intero progetto, incluso ciò che ognuno di loro ha scritto? Oppure dovrebbero entrambi guardare il codice dell'altro? Posso vedere vantaggi a ciascuno.
Infine, cosa succede con i problemi riscontrati durante la revisione del codice? Sono stati notati e quindi rispediti al programmatore? O il recensore dovrebbe annotarli e quindi correggerli? O è mai ok per fare un cambiamento e nemmeno dire al programmatore in particolare cosa hai trovato?