Questo documento fornisce istruzioni su come esaminare una modifica. Tieni presente i seguenti criteri per una modifica valida:
- Questa modifica rientra nello scopo dichiarato di questo progetto?
- Questa modifica è valida nell'architettura esistente del progetto?
- Questa modifica introduce difetti di progettazione che potrebbero causare problemi in futuro?
- Questa modifica segue le best practice stabilite per questo progetto?
- Questa modifica è un buon modo per eseguire la funzione descritta?
- Questa modifica introduce rischi di sicurezza o instabilità o interrompe le app esistenti?
Per ulteriori informazioni sull'esame di una modifica, consulta la sezione Esaminare una modifica della documentazione di Gerrit.
Se ti viene assegnato il ruolo di revisore per una modifica, dovresti ricevere un'email con un link alla modifica. Segui questi passaggi per esaminare la modifica:
Fai clic sul link alla modifica nell'email. Viene visualizzata la schermata Informazioni sulla modifica di Gerrit. Questa schermata fornisce una descrizione della modifica, identifica il collaboratore della modifica e tutti i revisori e fornisce un elenco dei file associati alla modifica.
Leggi la descrizione della modifica per comprenderne l'obiettivo.
Fai clic su un file per esaminare le modifiche apportate al file. Il file si apre in una schermata di confronto a due riquadri con il file esistente a sinistra e il file con le modifiche proposte a destra. Per esaminare il file:
Utilizza il mouse per evidenziare (fai clic e trascina) la parte di codice per cui hai un commento o una domanda. Viene visualizzato il messaggio "Premi c per commentare".
Premi c. Viene visualizzata una casella di commento in bozza.
Inserisci il commento o la domanda e fai clic su SALVA.
Fai clic sul numero univoco della modifica (in alto a sinistra nella schermata) per tornare alla schermata Informazioni sulla modifica.
Continua a esaminare ogni file della modifica.
Al termine dell'esame di tutti i file della modifica, torna alla schermata Informazioni sulla modifica e fai clic su RISPONDI. Viene visualizzata la schermata Voti per i requisiti di invio.
Scegli uno dei seguenti punteggi per la modifica:
- +1 Mi sembra tutto a posto, ma qualcun altro deve approvare.
0 Nessun punteggio
-1 Preferirei che non venisse inviato così com'è, ci sono cose da correggere.
-2 Questa modifica non deve essere inviata
Fai clic su RISPONDI per inviare i commenti e il punteggio allo sviluppatore della modifica.
Se ci sono problemi da risolvere, lo sviluppatore li risolve e ti chiede di esaminare le correzioni. In caso contrario, hai terminato l'esame della modifica.