[IV-23-24] Objetivo-2

[AlvaroSanPal95]

Sobre la estructura del repositorio

• [x] ¿He seguido las mejores prácticas en el nombre de las clases y ficheros y disposición de los mismos?
• [x] ¿He mirado tanto reglas generales (para cualquier repo) como reglas específicas para el lenguaje con el que se está trabajando?

Sobre el planteamiento

• [x] ¿Se han planteado una serie de issues, asignados a las historias de usuario pertinentes, que clarifiquen qué es lo que necesitan los usuarios e identifiquen las diferentes partes del problema?
• [x] ¿Los issues representan un problema, y no una tarea?

Sobre el análisis del problema

• [x] ¿Se ha documentado qué análisis se ha hecho sobre el dominio para decir lo que se ha creado?
• [x] ¿Se ha documentado por qué se ha elegido que lo creado sea un objeto valor, una entidad o un agregado?
• [x] ¿He propuesto un producto mínimamente viable, que en muchos casos será un solo objeto valor que no dependa de ningún otro (y que sea la base de muchos otros)?.

Sobre el código

• [x] ¿Todos los mensajes de commit explican el cambio, y no se limitan a repetir el nombre del fichero que se ha cambiado?
• [x] ¿Los mensajes de commit siguen el formato estándar y buenas prácticas?
• [x] ¿Se ha hecho una revisión real del código para comprobar que todos los atributos y funciones creadas están respaldadas por una HU?
• [x] ¿Todos los issues creados están relacionados con a una HU?
• [x] ¿Ha asignado el propietario/la propietaria del repositorio todos los issues planteados al milestone correspondiente?
• [x] ¿Todos los cambios en el código están asignados a un issue al que se referencia en un commit?
• [x] ¿Se ha asignado al mismo milestone el PR que se ha hecho?
• [x] ¿Es el milestone sobre el que estoy trabajando un producto mínimamente viable? ¿O tengo que solicitar al product manager que precise de qué producto se trata?
• [x] ¿Se ha comprobado que el código entregado sea sintácticamente correcto?

[JJ]

Porfa @shvtwp @jaimegm96 echadle un vistazo con cierta urgencia, que ya veis que va un poco atrasado. @migueruiz, muchas gracias.

[JJ]

En este objetivo hay, sobre todo, que seguir una metodología: crear unos issues que representen una serie de problemas, y tratar de resolverlos mediante el código. Has hecho en el mismo commit el fichero yaml y el código, sin ninguna referencia a ningún issue. Es imposible saber si es correcto o no. Tendrías que borrar ese código y comenzar de nuevo con una serie de issues que plantearan los diferentes problemas, e hicieran referencia a la historia de usuario #2. Te rogaría también que lo hicieras con cierta celeridad, para no bloquear el avance de tu compalero @smallPingu

[JJ]

Por favor, revisad esto ya, ayer mejor que hoy. No queda ya mucho tiempo para hacer el resto de los objetivos, ni siquiera para la convocatoria extraordinaria.

[AlvaroSanPal95]

Buenas @JJ ; Tenía razón en que solo hacía un commit con todos los cambios dentro, no respetando las buenas prácticas..

He eliminado los archivos con un commit, y luego he pusheado varios, haciendo referencia a que issue cierran, de las creadas en el repositorio del compañero.

Adicionalmente he creado una issue nueva para tener en cuenta el problema del .gitignore al tener elegido el lenguaje de programación, aspecto del que no me había acordado.

Le rogaría revisase los commits nuevos, cerrando algunos de los issues, y también ha los issues en sí para saber si los he creado adecuadamente y referenciando correctamente a las historias de usuario u otros issues.

Gracias de antemano por su tiempo.

[smallPingu]

@JJ ¿Sigo trabajando por donde lo dejó mi compañero?

[JJ]

Salvo que te haya dado permiso al crear el PR de edición en su fork, no creo que puedas. Prueba a editar alguno de los ficheros online; si puedes, puedes continuar, si no, tienes que cerrar y crear un nuevo PR.