10 Agosto - Code review #1

En esta sección se atenderán las dudas relacionadas con sistemas de gestión de la calidad, procedimientos, buenas prácticas, normas y todo lo relacionado con aseguramiento de la calidad

Moderator: julianmartinez16

Post Reply
User avatar
xacarana
Site Admin
Posts: 1028
Joined: Fri Jan 15, 2016 6:13 pm

10 Agosto - Code review #1

Post by xacarana » Thu Mar 09, 2017 6:44 am

  • Se debe entregar informe escrito (Puede ser en hojas reciclables)
  • Debe ser una revisión objetiva, profesional
  • Al equipo que se revise debe tener un mínimo de 20 commits, con sus respectivos mensajes explicativos
  • Se deben revisar muy bien los checklist y validarlos antes de revisar el diseño
  • Se debe realizar diseño y luego código
  • Incluir el diseño revisado y el checklist de diseño aprobado
  • Incluir el código y el checklist de código aprobado
  • Se deben poner una lista de errores encontrados (El que revisa)
  • Se debe poner una lista de mejoras (El que revisa)
  • Se debe incluir una lista de detalles técnicos (El que revisa)
Lista de revisión

Ej: Pablo revisa a Alejandro, Alejandro revisa a Otavio

Image
Andrés Bedoya Tobón
Profesor
"I only smile in the dark, I only smile when it's complicated" Raybiez

User avatar
xacarana
Site Admin
Posts: 1028
Joined: Fri Jan 15, 2016 6:13 pm

Re: Próximo code review

Post by xacarana » Tue Apr 18, 2017 7:09 am

Revisión 10 de agosto

:!: :!: :arrow: PRÓXIMA CLASE, mínimo 20 commit nuevos con avance evidente, checklist de diseño mínimo 20 items, checklist de código mínimo 20 items con las columnas (Si, No, No aplica). Traer listo el diseño de un nuevo componente y la implementación del código :!: :!:

:!: :!: Checklist diligenciados completamente, para poder arrancar el code review, se revisa al empezar la próxima clase :!: :!:
  1. Entregar informe escrito de lo revisado en clase
  2. Publicar las recomendaciones en el post del compañero, se recomienda un paralelo en los checklist (antes - después)
  3. Realizar la propuesta de 5 nuevos items por cada checklist
  4. Aprovechar el tiempo de clase para hablar, para avisar de alertas del proyecto a las cuales se les debe dar prioridad
  5. Asignar una nota cuantitativa del avance del proyecto para el code review
  6. Realizar seguimiento de los ajustes sugeridos, de tal forma que queden resueltos para el segundo code review
Andrés Bedoya Tobón
Profesor
"I only smile in the dark, I only smile when it's complicated" Raybiez

User avatar
Andres Morales
Posts: 41
Joined: Mon Jul 17, 2017 8:38 am

Re: Para tener en cuenta al realizar el code review

Post by Andres Morales » Mon Aug 07, 2017 8:30 am

Profesor, tengo unas dudas

1. ¿El informe escrito es sobre el code review que se haga en clase?
2. ¿Los checklist, es la lista de como vamos a revisar el proyecto? lo que usted nos había dicho, por ejemplo: las variables que empiecen en minúscula, nombres nanotecnicos.
3. El diseño, es el diseño de la aplicación? diagramas de clase, datos, de arquitectura, wireframes, etc?
4. ¿Qué se debe de llevar listo para la clase del code review?

User avatar
xacarana
Site Admin
Posts: 1028
Joined: Fri Jan 15, 2016 6:13 pm

Re: 10 Agosto - Code review #1

Post by xacarana » Tue Aug 08, 2017 7:23 am

  1. ¿El informe escrito es sobre el code review que se haga en clase?
    Si, el informe es la recomendación a la revisión realizada en clase o antes de la clase del diseño y del código. Se debe firmar y entregar documento físico.
  2. ¿Los checklist, es la lista de como vamos a revisar el proyecto? lo que usted nos había dicho, por ejemplo: las variables que empiecen en minúscula, nombres nanotecnicos.
    Si, son dos, checklist de diseño y checklist de código, es personal de cada proyecto y debe estar diligencia antes de la revisión, si hay un item que no se cumple, se devuelve y la revisión se para hasta que se corrija el error.
  3. ¿El diseño, es el diseño de la aplicación? diagramas de clase, datos, de arquitectura, wireframes, etc?
    Se deben realizar mínimo dos diseño, según los requerimientos de la aplicación. (Diagrama de clases, diagrama de secuencia,... los wireframes se incluyen en un diagrama de flujo lo más completo posible) El diagrama debe ser una ayuda para el proyecto, no realice diseño por cumplir.
  4. ¿Qué se debe de llevar listo para la clase del code review?
    Tarea del backlog lista y funcionando.
Andrés Bedoya Tobón
Profesor
"I only smile in the dark, I only smile when it's complicated" Raybiez

Post Reply