Revisión por pares: Difference between revisions
Jump to navigation
Jump to search
| Line 36: | Line 36: | ||
== Preparar la revisión == | == Preparar la revisión == | ||
La efectividad de la revisión depende en parte de la preparación: | La efectividad de la revisión depende en parte de la preparación: | ||
* El autor debe asegurarse de que el código compila, pasa los tests y tiene comentarios suficientes antes de solicitar la revisión. | |||
El autor debe asegurarse de que el código compila, pasa los tests y tiene comentarios suficientes antes de solicitar la revisión. | * El revisor debe tener acceso al contexto (la historia de usuario, la spec o los criterios de aceptación) para poder juzgar si la implementación es correcta, no solo si está bien escrita. | ||
El revisor debe tener acceso al contexto (la historia de usuario, la spec o los criterios de aceptación) para poder juzgar si la implementación es correcta, no solo si está bien escrita. | * Se recomienda limitar el tamaño de los ''pull requests'' a lo que puede revisarse en una sesión de 30-60 minutos. Los ''pull requests'' muy grandes reducen la calidad de la revisión. | ||
Se recomienda limitar el tamaño de los ''pull requests'' a lo que puede revisarse en una sesión de 30-60 minutos. Los ''pull requests'' muy grandes reducen la calidad de la revisión. | |||
== Revisión por pares del código generado por IA == | == Revisión por pares del código generado por IA == | ||