Obtención de comentarios con solicitudes de incorporación de cambios

Las solicitudes de incorporación de cambios admiten la revisión y combinación de código en un único proceso de colaboración. Una vez que un desarrollador agrega una característica o una corrección de errores, crea una solicitud de incorporación de cambios para comenzar el proceso de combinar los cambios en la rama ascendente. A continuación, otros miembros del equipo tienen la oportunidad de revisar y aprobar el código antes de que finalice. Use solicitudes de incorporación de cambios para revisar los trabajos en curso y obtener comentarios sobre los cambios. Pero no hay ningún compromiso para combinar los cambios. Un propietario puede abandonar una solicitud de incorporación de cambios en cualquier momento.

Haz revisar el código

La revisión de código realizada como parte de una solicitud de incorporación de cambios no es solo para encontrar errores obvios; eso es lo que son las pruebas. Una buena revisión de código detecta problemas menos obvios que podrían dar lugar a problemas costosos más adelante.

Las revisiones de código ayudan a proteger al equipo de combinaciones incorrectas y compilaciones interrumpidas que sapen la productividad del equipo. Revisa los problemas detectados antes de la combinación, protegiendo las ramas importantes frente a cambios no deseados.

Las revisiones de código también fomentan y refuerzan la colaboración y la comunicación entre los desarrolladores. Y el equipo obtiene un historial claro de todos los cambios realizados entre la rama principal y las ramas de funciones.

Fomente el intercambio de experiencia y la difusión de estrategias de resolución de problemas utilizando una amplia variedad de revisores en las revisiones de código. La difusión de habilidades y conocimientos hace que el equipo sea más fuerte y resiliente.

Dar excelentes comentarios

Las revisiones de alta calidad comienzan con comentarios de alta calidad. Las claves para una excelente retroalimentación en un pull request son:

• Haga que las personas adecuadas revisen la solicitud de incorporación de cambios.
• Asegúrese de que los revisores sepan lo que hace el código.
• Proporcione comentarios accionables y constructivos.
• Responder a los comentarios de forma oportuna.

Al asignar revisores a una solicitud de incorporación de cambios, asegúrese de seleccionar el conjunto correcto de revisores. Los revisores deben saber cómo funciona el código, pero también incluir a los desarrolladores que trabajan en otras áreas para que puedan compartir sus ideas.

Proporcione una descripción clara de los cambios y proporcione una compilación del código que tiene la corrección o característica que funciona en él. Los revisores deben hacer un esfuerzo para proporcionar comentarios sobre los cambios con los que no están de acuerdo. Identifique el problema y proporcione sugerencias específicas sobre lo que podría hacerse de forma diferente. La retroalimentación tiene una intención clara y es fácil para el propietario del pull request entenderla.

El propietario de la solicitud de incorporación de cambios debe responder a comentarios, aceptar sugerencias o explicar por qué rechaza aplicarlos. Algunas sugerencias son buenas, pero podrían estar fuera del ámbito de la solicitud de incorporación de cambios. Tome estas sugerencias y cree nuevos elementos de trabajo y ramas de características independientes de la solicitud de incorporación de cambios para realizar esos cambios.

Protección de ramas con directivas

Hay algunas ramas críticas en un repositorio en las que los equipos dependen de que siempre estén en buena forma, como la main rama. Teams puede requerir solicitudes de incorporación de cambios para realizar cualquier cambio en estas ramas con plataformas como GitHub y Azure DevOps. Los desarrolladores que insertan cambios directamente en las ramas protegidas tendrán sus inserciones rechazadas.

Agregue condiciones adicionales a las solicitudes de incorporación de cambios para aplicar un mayor nivel de calidad del código en ramas clave. Una compilación limpia del código combinado y la aprobación de varios revisores son algunos requisitos adicionales que a menudo se emplean para proteger las ramas clave.

Aprende más

GitHub tiene una amplia documentación sobre cómo proponer cambios en el trabajo con solicitudes de incorporación de cambios.

Obtenga más información sobre cómo proporcionar comentarios excelentes en las revisiones de código y usar plantillas de solicitud de incorporación de cambios para proporcionar instrucciones a los revisores. Azure DevOps también ofrece una experiencia enriquecida de pull requests que es fácil de usar y se adapta a las necesidades.