Revisión de PR
Revisa la superficie de cambios de la rama, no todo el repositorio.
- Establece el diff:
git diff <base>...HEAD(ygit diff <base>cuando el árbol de trabajo contiene cambios sin confirmar que pertenecen a la rama). La rama base la indica el solicitante; si no se especifica, se utilizamain. - Revisa cada fragmento modificado para verificar su corrección: resultados incorrectos en entradas límite, rutas de error no manejadas y comportamiento que contradice el nombre de la función o sus llamantes.
- Verifica la cobertura de pruebas para el comportamiento modificado: si hay lógica nueva sin una prueba que ejercite sus casos límite, se reporta como hallazgo.
- Revisa si el diff ha dejado documentación desactualizada: afirmaciones en README o documentación que eran ciertas en la rama base y son falsas después del cambio.
- Reporta cada defecto encontrado, cada uno con su severidad; incluye hallazgos menores e informativos. No reportes defectos en archivos que la rama no ha modificado; esos están fuera del alcance de la pull request.