vaniushar / cdmx-2018-01-fe-markdown Goto Github PK

Las funciones de tu archivo index.js en la mayoría de los casos son síncronos, recuerda que las funciones deben ser invocadas desde la función principal mdlinks, así tendrás mejor control sobre el return de cada función y el uso de las callbacks desde mdlinks te servirán demasiado.
En la línea 35 del archivo index.js estas ejecutando una función después del return, nunca se ejecutará.
El archivo app.js no veo que se esté utilizando.
Tampoco realizaste issues, ni milestones, puedes hacer tus issues desde el tablero de github, "Projects" es tu amigo!

¡Sigue esforzándote como hasta ahora, mucho éxito! :)

Vania:

Me gusto mucho que tuvieras separadas tus carpetas, felicidades por eso.
En el archivo app.js creo que nunca se ejecuta, deberías checar eso.
Me gusta que comentes tu código, pero tal vez pudieras hacerlo en español.

¡Muchas felicidades por tu alcance Vania, sigue asi!

Vania:
El Index.js principal debe de estar afuera de la carpeta de javascript debido a que es el primero que busca por defecto el navegador y al no encontrarlo en algún momento te traerá complicaciones.

Me gustaría que me explicaras porque tienes una carpeta de archivos independiente con el nombre de covergae, gracias..

Una de las cosas que note es que tú requieres la función mdlinks en tus test, sin embargo nunca la mandas llamar desde tu app.js, me parece que tendrias que separar esta parte.

Al generar los comentarios en tu código estas mezclando el español con el ingles. Es algo que podrías modificar, de esta manera se verá más unificado.

En la función principal mdLinks tu tienes la opción de utilizar un operador ternario para generar la comparativa inicial de los archivos y de pendiendo de la respuesta lanzar el mismo console.log, esto te permitirá disminuir las lineas de código y que el mismo se más entendible.
Lo mismo estaría aplicando para la función fs.readfile, ya que note que usas el if, pero nunca usas el else.

Tanto en la función stringTagCleanner como en la función stringCleaner se esta usando map para iterar sobre los arreglos, esto provoca que se repita el código, sería una buena idea juntar esta parte para evitar la repetición del código.

@VaniushaR

Hola Vania.

Dejaré algunos comentarios sobre tu proyecto.

• Te falta documentar en el Readme.
• Recuerda que un Readme tambien funciona como instructivo para el usuario. ¿ Que es este proyecto? ¿ Como funciona? ¿ Como lo uso? ¿ Necesito descargar algo?
• La carpeta de "coverage" creo que era mejor que no se subiera a github.
• En tu gitignore tienes package.lock.json. Pero debería ser package-lock.json.
• Tienes un archivo spec.js y ese debería de estar en la carpeta de test.
• Muy bien que todo lo pongas en inglés
• No estas usando eslint.
• No pude correr tu proyecto, no me funcionan los comandos que se piden en el Readme. Me dice que el comando no fue encontrado.

Vania, si hay algo en lo que te podamos apoyar, no dudes en avisarnos. Recuerda que el equipo de formación estamos para ustedes. Sigue con esa gran actitud y compromiso! 😃