Closed ErikaVorozco closed 7 months ago
Por favor, seguid a rajatabla las buenas prácticas en los mensajes de commit. Recordad que todos tienen que referirse a un issue.
Tampoco está enlazado correctamente desde el cuerpo del PR, hay que poner un espacio antes del #
Es posible que en vez de crear una clase para la id del empleado la podamos implementar en la clase que cree yo anteriormente?
Con los commit tenemos que intentar que sean "útiles", es decir, que con el título sepamos a qué hace referencia (el número del issue en cuestión) y luego en el cuerpo del mismo explicar la motivación del cambio. No explicar el cambio en sí, que eso se puede ver viendo el propio código, sino el porqué del cambio.
@maguilerafr ¿crees que lo hecho corresponde a lo pedido por @Lolic3 en su comentario?
@maguilerafr ¿crees que lo hecho corresponde a lo pedido por @Lolic3 en su comentario?
Yo entiendo que sí, que la idea es que más adelante se pueda unir. Quizás hay que especificar eso, que este código se añadirá en una MS posterior a la clase que creó @Lolic3
@maguilerafr ¿crees que lo hecho corresponde a lo pedido por @Lolic3 en su comentario?
Yo entiendo que sí, que la idea es que más adelante se pueda unir. Quizás hay que especificar eso, que este código se añadirá en una MS posterior a la clase que creó @Lolic3
Si es así, comentadlo así mismo en el PR en vez de aprobarlo, ¿no?
Ya hemos cambiado el PR, échale un vistazo cuando puedas @JJ por favor
closes #31 feat #31 Tal como menciona en la HU1, he creado la función employeeId para luego más adelante, implementarla en la clase employee de mi compañero @Lolic3.