GenesisMauries / markdown

Markdown
0 stars 0 forks source link

Refactorizar forEach por for #22

Open BrisiaCastrejon opened 6 years ago

BrisiaCastrejon commented 6 years ago

Hola Gen: Considero que podrías cambiar tus for por forEach en tus funciones: onlyLinks y runArray . También creo que podrías eliminar esta funcion : const checkRute = (ruta) => { const rute = (path.isAbsolute(ruta) === true) ? ruta : path.resolve(ruta); console.log(rute); }; checkRute('./README.md'); pues en realidad no está haciendo nada más que corregir la ruta de archivo, y tu línea 14: fs.readFile('README.md', 'utf-8', callback = (err, md) => ya está recibiendo la ruta del archivo. de lo demás me gusta que tengas funciones pequeñas y que este comentado tu código, esto último es algo que me llevo para la realización del mio.

GenesisMauries commented 6 years ago

Gracias Brisia, agradecería tu ayuda ya que pensaba pasar la función checkRute para ejecutarla como parámetro en readFile. :octocat: