[IS-23-24] paso nivel 2 tasapp

[AndreaMGG]

Creación fichero is.yaml + creación issue elección del lenguaje de programación

• ¿Todos los mensajes de commit explican el cambio, y no se
  limitan a repetir el nombre del fichero que se ha cambiado?
• ¿Los mensajes de commit siguen el formato estándar y buenas prácticas?
• ¿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?
• ¿Todos los issues creados están relacionados con a una HU?
• ¿Ha asignado el propietario/la propietaria del repositorio todos los
  issues planteados al milestone correspondiente?
• ¿Todos los cambios en el código están asignados a un issue al que se
  referencia en un commit?
• ¿Se ha asignado al mismo milestone el PR que se ha hecho?
• ¿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?
• ¿Se ha comprobado que el código entregado sea sintácticamente
  correcto?

closes #32,
closes #35,
closes #36,
closes #37,
closes #38,
closes #39,
closes #45,
closes #46,
closes #47,
closes #48,
closes #49,
closes #50,
closes #51,
closes #52,
closes #53,
closes #55,
closes #57,
closes #59,
closes #61,
closes #62,
closes #64,
closes #65

[AndreaMGG]

@JJ listo para revisión definitiva

[JJ]

¿Veis como hay que revisarlo todo?

[JJ]

"Español" es un idioma, no un lenguaje.

[JJ]

Por favor, dadle un vistazo a esto.

[AndreaMGG]

Sobre la estructura del repositorio

• ¿He seguido las mejores prácticas en el nombre de las clases y ficheros y
  disposición de los mismos?
• ¿He mirado tanto reglas generales (para cualquier repo) como reglas
  específicas para el lenguaje con el que se está trabajando?
  Sobre el planteamiento
• ¿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?
• ¿Los issues representan un problema, y no una tarea?
  Sobre el análisis del problema
• ¿Se ha documentado qué análisis se ha hecho sobre el dominio para decir lo
  que se ha creado?
• ¿Se ha documentado por qué se ha elegido que lo creado sea un objeto valor,
  una entidad o un agregado?
• ¿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
• ¿Todos los mensajes de commit explican el cambio, y no se
  limitan a repetir el nombre del fichero que se ha cambiado?
• ¿Los mensajes de commit siguen el formato estándar y buenas prácticas?
• ¿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?
• ¿Todos los issues creados están relacionados con a una HU?
• ¿Ha asignado el propietario/la propietaria del repositorio todos los
  issues planteados al milestone correspondiente?
• ¿Todos los cambios en el código están asignados a un issue al que se
  referencia en un commit?
• ¿Se ha asignado al mismo milestone el PR que se ha hecho?
• ¿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?
• ¿Se ha comprobado que el código entregado sea sintácticamente
  correcto?

[JJ]

¿Habéis revisado todos los issues del milestone a ver si habéis creado código para resolverlos todos?

[maguilerafr]

¿Habéis revisado todos los issues del milestone a ver si habéis creado código para resolverlos todos?

Hemos revisado y creemos que sí

[JJ]

¿Y habéis puesto en el cuerpo del PR los issues que se van a cerrar cuando se mergee?

[JJ]

De la comprobación de sintaxis:

Traceback (most recent call last):
  File "/Users/jjmerelo/.pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/Users/jjmerelo/.pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/Users/jjmerelo/Asignaturas/MPDA-IS/T-asapp/recipeers/recetario.py", line 8, in <module>
    from tipo_receta import Tipo_receta
ImportError: cannot import name 'Tipo_receta' from 'tipo_receta' (/Users/jjmerelo/Asignaturas/MPDA-IS/T-asapp/recipeers/tipo_receta.py)

[maguilerafr]

De la comprobación de sintaxis:

Traceback (most recent call last):
  File "/Users/jjmerelo/.pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/Users/jjmerelo/.pyenv/versions/3.9.13/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/Users/jjmerelo/Asignaturas/MPDA-IS/T-asapp/recipeers/recetario.py", line 8, in <module>
    from tipo_receta import Tipo_receta
ImportError: cannot import name 'Tipo_receta' from 'tipo_receta' (/Users/jjmerelo/Asignaturas/MPDA-IS/T-asapp/recipeers/tipo_receta.py)

Creíamos que habíamos corregido ese fallo, ya lo hemos hecho definitivamente.

[maguilerafr]

¿Y habéis puesto en el cuerpo del PR los issues que se van a cerrar cuando se mergee?

Ya hemos hecho el listado de los issues a cerrar y lo hemos puesto en la descripción del PR.
Son estos: closes #32, #35, #36, #37, #38, #39, #45, #46, #47, #48, #49, #50, #51, #52, #53, #55, #57, #59, #61, #62, #64, #65

[JJ]

Adicionalmente

poetry run pyflakes recipeers/*.py
recipeers/preparacion.py:7:1: 'ingrediente.Ingrediente' imported but unused
recipeers/recetario.py:8:1: 'tipo_receta.Tipo_receta' imported but unused

[JJ]

No sé donde visteis lo de que se podían separar por comas, pero conviene que os acostumbréis a usar la fuente. En este manual explica como hacerlo, incluso con issues de otro repositorio (si se tiene permiso, claro)

[maguilerafr]

No sé donde visteis lo de que se podían separar por comas, pero conviene que os acostumbréis a usar la fuente. En este manual explica como hacerlo, incluso con issues de otro repositorio (si se tiene permiso, claro)

ok, gracias, le echo un ojo