Proceso de Pair Review - novaDepto/Nova GitHub Wiki
| Nombre | Rol |
|---|---|
| Charlie | Dueño del proceso |
| Lisieux | Autor |
| Raymundo | Autor |
- Verificar el código realizado antes de mergearlo a develop
- Asegurar la calidad de los productos de trabajo del proyecto
- Pull Request a develop
- Los proyectos usan 'pylint' y 'eslint' para detectar errores en el código y asegurar que siga estándares de codificación, por lo que la verificación de estos productos de trabajo usando debe atenerse solamente a aspectos que dichas herramientas no cubren.
- Los equipos pueden añadir o cambiar componentes del entorno de verificación recomendado en la Guía del Entorno de Verificación dependiendo de sus necesidades.
| Fase | Actividades | Encargado | Áreas CMMI |
|---|---|---|---|
| Preparación | Comunica al equipo que hay un Pull Request en el repositorio. | Autores del Pull Request | VER |
| Decide quiénes realizarán la Pair Review. | |||
| Entrega a la pareja los archivos necesarios para revisar el Pull Request (Consulta las Precondiciones en la Guía del Entorno de Verificación). |
|||
| Verificación | Verifica el código siguiendo el Reporte de Pair Review. | Pair Reviewer | VER |
| Documenta los defectos encontrados en el Log de defectos. | |||
| Comunicación | Comunica a la pareja solicitante los resultados de la Pair Review. | Pair Reviewer | VER |
| a) Aprobado = Realiza el merge de la rama a develop. | |||
| b) Rechazado = Comenta en el Pull Request las correcciones que se le deben hacer al código y pega el link del Reporte de Pair Review. | |||
| Seguimiento | Si el Pull Request fue rechazado, entonces se podrá verificar nuevamente siempre y cuando se hayan realizado las correcciones de la revisión aterior. | Pair Reviewer | VER, PMC |
- Reporte de Pair Review
- Pull Request aprobado/rechazado
- Defectos encontrados por fase
versión 2.0