Questo documento fornisce istruzioni su come esaminare una modifica. Tieni presente 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 per il futuro?
- Questa modifica segue le best practice per questo progetto?
- Questa modifica è un buon modo per eseguire la funzione descritta?
- Questa modifica introduce rischi di sicurezza o instabilità o comporta il malfunzionamento di app esistenti?
Per ulteriori informazioni sulla revisione di una modifica, consulta Revisione di una modifica della documentazione Gerrit.
Se ti viene assegnato il ruolo di revisore per una modifica, dovresti ricevere un'email con un link alla modifica. Per esaminare ed esaminare la modifica:
Fai clic sul link dell'email alla modifica. Informazioni sulla modifica di Gerrit . Questa schermata fornisce una descrizione della modifica, identifica chi ha contribuito alla modifica e tutti i revisori e fornisce un elenco dei file associati alla modifica.
Leggi la descrizione della modifica per comprendere lo scopo della modifica.
Fai clic su un file per esaminare le modifiche che contiene. Il file si apre in due riquadri schermata di confronto con il file esistente a sinistra e il file con modifiche a destra. Per esaminare il file:
Utilizza il mouse per evidenziare (fai clic e trascina) la porzione di codice per la quale se hai un commento o una domanda. Il messaggio "Premi C per commentare" .
Premi C. Viene visualizzata una casella di commento alla bozza.
Inserisci il commento o la domanda e fai clic su SALVA.
Fai clic sul numero univoco della modifica (lato in alto a sinistra della schermata) per tornare alla schermata Modifica informazioni.
Continua a esaminare ogni file della modifica.
Una volta esaminati tutti i file della modifica, torna alla schermata Modifica informazioni e fai clic su RISPONDI. Viene visualizzata la schermata Invia voti per i requisiti.
Scegli uno dei seguenti punteggi per la modifica:
- +1 Mi sembra perfetto, ma deve essere approvata anche da qualcun altro.
0 Nessun punteggio
-1 Preferisco che non venga 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 corregge e ti chiede di farlo. rivedi le correzioni. In caso contrario, avrai terminato di esaminare la modifica.