En este documento, se proporcionan instrucciones para revisar un cambio. Ten en cuenta los siguientes criterios para un cambio válido:
- ¿El cambio se ajusta al propósito declarado de este proyecto?
- ¿El cambio es válido dentro de la arquitectura existente del proyecto?
- ¿El cambio introduce fallas de diseño que causarán problemas en el futuro?
- ¿El cambio respeta las prácticas recomendadas establecidas para este proyecto?
- ¿El cambio es una buena forma de realizar la función descrita?
- ¿El cambio introduce algún riesgo de seguridad o inestabilidad? ¿O hace fallar apps existentes?
Si quieres obtener información adicional, consulta la sección para revisar un cambio de la documentación de Gerrit.
Si se te asignó como revisor de un cambio, deberías recibir un correo electrónico con un vínculo a ese cambio. Sigue estos pasos para revisar el cambio:
Haz clic en el vínculo al cambio incluido en el correo electrónico. Aparece la pantalla Change Info de Gerrit. En esta pantalla, se brinda una descripción del cambio, se identifica a quién lo aportó y a todos los revisores, y se proporciona una lista de los archivos asociados con el cambio.
Lee la descripción del cambio para comprender su objetivo.
Haz clic en un archivo para revisar sus cambios. El archivo se abre en una pantalla de comparación de dos paneles, con el archivo existente a la izquierda y los cambios propuestos a la derecha. Para revisar el archivo, haz lo siguiente:
Destaca con el mouse (clic + arrastrar) la parte del código para la que tienes un comentario o una pregunta. Aparece el mensaje "Press c to comment".
Presiona c. Aparece un cuadro de borrador de comentario.
Ingresa tu comentario o pregunta y haz clic en SAVE.
Haz clic en el número único del cambio (esquina superior izquierda de la pantalla) para regresar a la pantalla Change Info.
Sigue revisando cada archivo en el cambio.
Cuando termines de revisar todos los archivos en el cambio, regresa a la pantalla Change Info y haz clic en REPLY. Aparece la pantalla Submit requirement votes.
Elige una de las siguientes puntuaciones para el cambio:
- "+1" significa que tú lo apruebas, pero alguien más debe hacerlo también.
"0" significa sin puntuación.
"-1" significa que preferirías que no se enviara como está, ya que quedan cosas para corregir.
"-2" significa que este cambio no se debería enviar.
Haz clic en REPLY para enviar tus comentarios y puntuación al desarrollador del cambio.
Si hay problemas que se deben corregir, el desarrollador los corregirá y te solicitará que revises esas correcciones. De lo contrario, habrás finalizado la revisión del cambio.