[MAD-PT DE] Pilar Garcia #560
Conversation
Add gitignore
Add iterations from 1 to 3
Add bonus iterations
Merge develop with master
Add last line to gitignore file
Merge develop with master
|
Bieeen, vas mejorando mucho en el trabajo con las ramas (paso a no comentarte más el .gitignore, creo que ya lo has pillado) Sobre las ramas, tal y como dices, tuviste que sacar hotfx, pero... Eso no debería haber generado una nueva tag 1.0.1?? Paso a revisar el código Vale, has hecho buen uso de funciones de reponsabilidad única, ahora démosle una vueltecita de tuerca más. Deberías declarar las funciones arriba del código, y utilizarlas debajo. Declaraciones arriba, llamadas abajo, de tal forma que el que lea tu código, parezca que lea un libro, y si necesita comprender o ampliar un concepto, puede ir a revisar el apéndice. Si esto que acabo de decir no te resulta claro, te lo explico en persona. Líneas 26 y 31: por qué no directamente hacer return str.split("").reverse().join("")? Línea 55: No has tenido en cuenta el caso "Et"... habria que darle otra vuelta de tuerca si te parece para considerar ese caso. CORRIJO: ya he visto el toLowerCase() -> estoy pendiente del escrutinio jajaja Lian 89-90: por qué hacer un split para justo después hacer un join? Línea 94: no entiendo su uso, sería interesante si me lo explicas Good job! |
🎯Hasta dónde he llegado: 3 iteraciones + 2 bonus
💪Dificultad: media
🎓Comentarios:
He intentado crear funciones que pudiesen ser reutilizables. Para eliminar espacios en blanco y caracteres especiales de un string he estado investigando las regex pero no he conseguido una que hiciese las dos cosas a la vez, me cuesta mucho entender la sintaxis de esas expresiones, he recurrido a stackoverflow y otras referencias.
He utilizado git flow, una vez mergeada la primera feature en dev he creado la segunda sin hacer pull en mi local de dev por lo que cuando he mergeado la segunda he tenido que resolver conflictos, me lo esperaba pero queria ver como era la interfaz para resolverlos, me resulta un poco liosa, pero creo que lo he resuelto. Pero como consecuencia me ha quedado un arbol un poco lioso.
Hotfix para incluir linea en blanco en gitignore.