Después de abrir una solicitud de extracción, los revisores dejan comentarios que ayudan a mejorar el código antes de fusionarlo. Atender esas observaciones implica comprender cada comentario, hacer los cambios que se solicitan y asegurarse de que se ha abordado todo.
Comprender los comentarios de revisión
Los revisores pueden dejar comentarios generales, comentar líneas específicas y proponer cambios exactos como sugerencias. Las revisiones aparecen en la escala de tiempo de la solicitud de incorporación de cambios para poder seguir la discusión y ver qué comentarios todavía necesitan una respuesta. En función de la configuración del repositorio, un revisor también puede aprobar tu solicitud de extracción o solicitar cambios que debes realizar antes de fusionarla.
Antes de cambiar cualquier cosa, lea cada comentario para comprender su intención. Un comentario puede señalar un error, formular una pregunta, solicitar un enfoque diferente o sugerir una edición que pueda aceptar directamente.
Si tiene acceso a Copilot, puede ayudarle a interpretar los comentarios de revisión y proponer correcciones, lo que resulta útil cuando una solicitud de extracción tiene muchos comentarios que revisar.
Implementación de cambios y corrección de código
Al dirigir los comentarios, puede hacer lo siguiente:
- Aplique directamente el cambio sugerido por un revisor, que confirma la sugerencia en la rama.
- Realice modificaciones más amplias localmente e inserte nuevas confirmaciones en la rama.
La mayoría de los comentarios se resuelve actualizando el código e insertando nuevas confirmaciones en la rama de solicitud de incorporación de cambios. A medida que aborda los comentarios, marque las conversaciones como resueltas para que usted y los revisores puedan realizar un seguimiento de lo que se hace y lo que todavía necesita atención.
Cuando los revisores obligatorios hayan aprobado y ya no haya cambios solicitados, la solicitud de incorporación de cambios estará lista para proceder a la fusión. Dado que la solicitud de incorporación de cambios realiza un seguimiento de la rama, cada nueva confirmación actualiza la solicitud de incorporación de cambios y vuelve a ejecutar las comprobaciones automatizadas.
Trabajar con comentarios de forma eficaz
En solicitudes de incorporación de cambios más grandes o más analizadas, algunas prácticas le ayudarán a abordar los comentarios rápidamente:
- Aceptar sugerencias por lotes agregando los cambios sugeridos a un lote para que varios cambios aceptados se incluyan en una sola confirmación de cambios en lugar de una confirmación de cambios por sugerencia.
- Resolver de forma proactiva cualquier conflicto de fusión para poder fusionar rápidamente una vez resuelta la revisión de la solicitud de extracción.
- Solicita revisiones de nuevo después de realizar cambios importantes, para que los revisores sepan que la solicitud de extracción está lista para volver a revisarla.
- Comprender las revisiones obligatorias. Cuando una rama requiere aprobaciones o la aprobación del propietario del código, una revisión de “solicitud de cambios” o una aprobación descartada pueden bloquear la fusión hasta que se resuelvan.
- Realice un seguimiento de los comentarios que quedan fuera del alcance abriendo una incidencia que remita al comentario, en lugar de ampliar la solicitud de incorporación de cambios.
Herramientas para ayudarle a resolver las revisiones
No tiene que revisar los comentarios manualmente usted solo:
-
Consulte la solicitud de incorporación de cambios localmente o ábrala en GitHub Codespaces para reproducir problemas y probar correcciones antes de insertarla.
-
Abordar los resultados de seguridad generados durante la revisión. Las alertas de análisis de código de los cambios aparecen en la solicitud de incorporación de cambios para poder corregirlas antes de combinarlas.
-
Uso Copilot para interpretar comentarios, responder preguntas, proponer correcciones y resolver conflictos de combinación con agentes.