Una solicitud de extracción es más fácil de revisar cuando su propósito es claro y su diferencia cuenta una historia coherente. Dividir el trabajo no relacionado reduce la posibilidad de que un cambio de comportamiento importante quede oculto durante el formateo o la limpieza.
Elija un límite útil
- Separe la refactorización mecánica de los cambios de comportamiento cuando eso haga que el resultado sea más fácil de verificar.
- Mantenga una solicitud de extracción lo suficientemente pequeña como para revisarla, pero no la divida hasta el punto de que cada parte no tenga sentido por sí sola.
- Describe las dependencias entre los cambios y el orden en el que deben realizarse.
Haga explícita la ruta de revisión
Explique el resultado del usuario o del sistema, la elección de implementación importante y cómo la probó. Vincular cuestiones relacionadas o trabajos de seguimiento en lugar de ampliar silenciosamente el alcance actual.
Antes de solicitar una revisión, lea la diferencia como alguien que no está familiarizado con la tarea. Elimine el ruido generado y marque cualquier pieza intencionalmente incompleta.