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 se 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.

Obtención de revisiones de código

La revisión del código realizada como parte de una solicitud de incorporación de cambios no sirve solo para encontrar errores obvios, ya que para eso sirven las pruebas. Una buena revisión del 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 las combinaciones incorrectas y las compilaciones rotas que disminuyen la productividad del equipo. Las revisiones detectan problemas antes de la combinación, protegiendo las ramas importantes de 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 características.

Realice una polinización cruzada de la experiencia y distribuya estrategias de solución de problemas mediante el uso de una amplia gama de revisores en sus revisiones de código. La diferenciación de conocimientos y aptitudes hace que el equipo sea más fuerte y resistente.

Realización adecuada de comentarios

Las revisiones de alta calidad comienzan con comentarios de alta calidad. Las claves para recibir comentarios excelentes en una solicitud de incorporación de cambios son:

• Pedir a las personas correctas que revisen la solicitud de incorporación de cambios.
• Asegurarse de que los revisores saben lo que hace el código.
• Proporcionar comentarios útiles y constructivos.
• Responda 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 una compilación del código que incluya la corrección o característica funcionando en ella. Los revisores deben esforzarse por proporcionar comentarios sobre los cambios con los que no están de acuerdo. Deben identificar el problema y realizar sugerencias específicas sobre lo que podría hacerse de forma diferente. Estos comentarios tienen una intención clara y son fáciles de entender para el propietario de la solicitud de incorporación de cambios.

El propietario de la solicitud de incorporación de cambios debe responder a comentarios, aceptar sugerencias o explicar por qué rechaza aplicarlas. 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 que los equipos confían que estarán siempre en buenas condiciones, como la rama main. Los equipos pueden requerir solicitudes de incorporación de cambios para realizar cualquier cambio en estas ramas con plataformas como GitHub y Azure DevOps. Los desarrolladores que inserten cambios directamente en las ramas protegidas verán rechazados sus envíos.

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

Más información

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 cómo usar plantillas de solicitud de incorporación de cambios para proporcionar instrucciones a los revisores. Azure DevOps también ofrece una experiencia de solicitud de incorporación de cambios enriquecida que es fácil de usar y se escala según sea necesario.