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

@esmerivero

• Falta documentar tu Readme.
• Recuerda que el Readme tambien funciona como un instructivo para el usuario, donde deberías contestar las preguntas. ¿ Que es el proyecto? ¿ Para que me sirve? ¿Como lo hago funcionar? ¿ Tengo que instalar algo? ¿ Que comandos debo correr?
• En la carpeta de test, hay un archivo pero está vacio.
• No usas eslintrc
• El else de la linea 98 del archivo cli.js pudiste ahorrartelo. Ya que solo evaluas un if.
• Podrías intentar usar for in, for of o forEach en lugar de for normales. Incluso hasta puedes usar .map .filter, etc. Cosas que te ayuden a hacer tu código mas pequeño.
• en la linea 42 de cli.js tienes un return, pero si hay un error el usuario no lo sabrá, podrías poner un console.log para arrojar el error.
• No pude correr tu proyecto, los comandos que vienen en el Readme no me los reconoce.

Esmeralda, 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! 😃

Esme, muchas felicidades por tu alcance y tu esfuerzo.

Me gusto que tengas tu carpeta de test, pero hubiera estado padre que tuviera algo.
También me encanta que hayas usado la parte de project y que tengas tus issues.
Creo que la linea 34 puede hacerse más simple, deberias checarlo.

De nuevo muchas felicidades por tu alcance y esfuerzo, Esme, buen trabajo.

La función mdlinks debe regresar un promise con un array de objects, con todos los links, estado y validación de los links

1. readFile
2. convertFile
3. validate
4. status

Esmeralda:
Me gustaría saber el motivo por el cual tu código se encuentra plasmado en el archivo cli.js
El acomodo de las carpetas debe ser congruente con tu estructura en general, por ello pienso que lo mejor sería que tu archivo index fuera el principal documento desde donde se está requiriendo el siguiente archivo, el nombre de este debe ser descriptivo para que se encuentre con mayor facilidad.

Tú código es entendible y claro con lo que solicita cada función. Sin embargo, la principal área de oportunidad es que repites mucho en cierta parte áreas de tu código, por ejemplo en la función stats y validate, se repite el for in y el map. Esto es algo que deberías estar cambiando.
En algunos casos presentas las funciones con calbacks, pero no en todas, es algo que podrías mejorar.

Felicidades por tú alcance.

Querida Esme,

Está evolucionado tu código a invocación mediante promesas y creo que es admirable ese alcance. Acá te tengo sólo unas sugerencias de mejora:

1. El else de la línea 33, puede ser prescindible, ya que sólo hay un if antes, por default si no entra en ese if, se ejecutará lo que está dentro de else.
2. En la 34, tienes un return con resolve( map, for in, if, fech,return content ) creo que eso podrías simplificarlo metiéndolo dentro de una función, a quien le pases el parámetro con la info obtenida en esa función, si es que trae, (creo que es content) para que te retorne sólo la ejecución de la función(parámetro) y sea más human redable y se puedan separar mejor las tareas de cada función.
   Es todo, excelente trabajo Esme ;)