Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Ya he dejado la función como estaba. Seguiré intentando resolver el problema en mi ordenador
- Loading branch information
Ya he dejado la función como estaba. Seguiré intentando resolver el problema en mi ordenador
47b0702
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hola @jmalcan
Si quieres se puede quitar Open. El fallo que ha dado es de los tests que comprueban que el resultado no cambia. Obviamente en este caso sí cambia. Pero puedo actualizar el test quitando Open si quieres. Ya me dices
47b0702
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
47b0702
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vale, ok. Ten en cuenta que si todas las funciones tienen tests, se puede comprobar automáticamente si ese cambio de quitar Open rompe algo por otro lado. Eso es algo fundamental en ingeniería de software. Pero está bien comprobar de manera manual y homogeneizar todo.
Para editar, como prefieras, puedes editar directamente el código en github (usando el lápiz de edit y luego hacer commit), o en tu ordenador y luego subir la versión actualizada de la función