En este commit, PR o MR, es fácil dejar comentarios sobre si el “cambio” que ocurrió está bien o no, pero detectar y comentar las partes que debieron haberse cambiado y no se cambiaron sigue pareciéndome difícil, incluso después de haber hecho code review durante bastante tiempo. Si no es un codebase con el que estoy muy familiarizado, todavía más.
Se me ocurren algunos métodos, pero ojalá hubiera más mecanismos. Me gustaría saber cómo lo han hecho ustedes.
- Al implementar: hacer que las partes que después deban cambiarse en conjunto queden cubiertas por el código de pruebas.
- Al redactar el PR: agregar en la plantilla del PR preguntas adecuadas para que la persona autora considere durante la revisión. (p. ej., ¿hay algún PR anterior con un propósito similar a este PR? Al compararlo con ese PR, ¿hay alguna parte que quizá se haya omitido en este PR?)
- Cada vez que en ese codebase ocurra algo de “debió haberse hecho, pero se omitió”, dejarlo registrado y actualizar la plantilla del PR con ese contenido como ejemplo.
- Al revisar el PR: añadir a la guía de revisión indicaciones para que la persona revisora busque partes faltantes, de forma similar a las preguntas anteriores.
Aún no hay comentarios.