florenciasilva / modulo-npm-markdown-links Goto Github PK

Tengo algunos comentarios sobre el código de tu archivo markdown:

• En varias partes estás usando let pero no es algo que vas a reescribir en el futuro. Por ejemplo

  modulo-npm-markdown-links/markdown.js

  Line 14 in 96cfdfb

  if (err) throw err;

  en esos casos sería más adecuado usar const

• Aquí podrías usar string literal o template strings

  modulo-npm-markdown-links/markdown.js

  Line 53 in 96cfdfb

  console.log('Broken' + ' ' + broken + '\n' +

• No llegué a ver ningún test 😢

¡HOLA!

¡Hola!

Leyendo tu código lo encontré lo siguiente:

• [✔ ] Funciones modularizadas.

• [ ✔] Uso de promesas

• [✔ ] Los comandos de línea son funcionales y devuelven los analisis de las URL.

Yo veo bastante concreto tu código y cumple con las caracteristicas que nos pedían. <3

Hola Flo.

En general veo un código muy limpio y sencillo.
En la documentación muy bien, me llevo algunas ideas para mi propio readme jeje.
En el código:

• En la linea 4 faltan unos ";".
• En la linea 13 haces la lectura del file system, quizá esa parte podría ir mejor en una función.
• Tienes algunos casos donde la identación no corresponde. Si usar Visual Code S puedes identar tu file, con shift+alt+f.
• Igual hay muchas variables declaradas con Let que pueden ser Const, no se si algunas sean necesarias en let, pero cambie a la azar un par y seguía corriendo n_nU.
• La función requestHTTP no tiene un return, seria problemático quizá a la hora de los test.

Creo que es todo, mucho éxito con tu siguiente alcance.