Sugerencias Tarea 1 #14
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Excelente código :), cumple correctamente con todos los requisitos solicitados en la tarea. De igual manera realizaste correctamente el uso de las tags, el manejo de Issues y utilizaste dos ramas en GitHub. Tu manejo de las expresiones regulares es increíble, sigue así 🥇. Solamente me gustaría darte dos recomendaciones, la primera es que te asegures de eliminar los saltos de linea al momento en el que extraes tu secuencia, esto lo puedes hacer con un rstrip. Por otro lado, considero que tus docstrings están muy bien hechos, sin embargo creo que seria de mucha ayuda que agregaras un poco mas de lineas de documentación para entender mejor la lógica que empleaste en algunos pasos.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Excelente código :), cumple correctamente con todos los requisitos solicitados en la tarea. De igual manera realizaste correctamente el uso de las tags, el manejo de Issues y utilizaste dos ramas en GitHub. Tu manejo de las expresiones regulares es increíble, sigue así 🥇. Solamente me gustaría darte dos recomendaciones, la primera es que te asegures de eliminar los saltos de linea al momento en el que extraes tu secuencia, esto lo puedes hacer con un rstrip. Por otro lado, considero que tus docstrings están muy bien hechos, sin embargo creo que seria de mucha ayuda que agregaras un poco mas de lineas de documentación para entender mejor la lógica que empleaste en algunos pasos.