Natalia Olmos PR MDLINKS

[NataliaOlmos]

No description provided.

[reloadercf]

Me gusta que tu proyecto resuelve el problema planteado, te recomiendo mejorar la definición de tus variables recuerda que tu código seria leído por algún reclutador que consulte tu portafolio de trabajo, otro aspecto a mejorar y tener cuidado con la lógica planteada en algunas funciones porque me percate que tu código se puede refactorizar y simplificar, algo que sin duda destaco es el correcto uso de variables.

[reloadercf]

Comentarios más formales

Suggested change

	## Haciendo pruebas de mdlinks
	## Haciendo pruebas de mdlinks

[reloadercf]

Complementa tu README con recursos para el usuario, (capturas de pantalla o videos) sobre la instalación y alguna documentación referente a tu librería.

[reloadercf]

Esta cool el uso de const para variables que no mutan su valor.

Suggested change

	const fs = require('fs')
	const fetch = require('node-fetch')
	const colors = require('colors')
	const fs = require('fs')
	const fetch = require('node-fetch')
	const colors = require('colors')

[reloadercf]

mejorar la definición

Suggested change

	function getContentString() {
	let index = process.argv.indexOf("--file")
	if (index < 0) return console.log("You need to use a valid uri flag --file")
	let uri = process.argv[index + 1]
	let string = fs.readFileSync(uri, 'utf8')
	return string
	}
	function getContentPath() {
	let index = process.argv.indexOf("--file")
	if (index < 0) return console.log("You need to use a valid uri flag --file")
	let path = process.argv[index + 1]
	let file = fs.readFileSync(uri, 'utf8')
	return file
	}

[reloadercf]

Definir mejor los parámetros recibidos

Suggested change

	function getLinks(string) {
	const getArray = ("Text: ", string.toString());
	let regExpression = /(http(s)?:\/\/.)?(www\.)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)/g;
	return getArray.match(regExpression);
	}
	function getLinks(file) {
	const getArrayLinks = ("Text: ", string.toString());
	let regExpression = /(http(s)?:\/\/.)?(www\.)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)/g;
	return getArrayLinks.match(regExpression);
	}

[reloadercf]

Tener un if vacío con comentario no es la mejor practica

Suggested change

	if (shouldShowTotals > -1) {
	// contar
	// muestro results
	}

[reloadercf]

Mejorar la descripción en ingles

Suggested change

	"description": "Proyecto mdlinks de Laboratoria",
	"description": "Proyecto mdlinks de Laboratoria",

[reloadercf]

recuerda que el apartado de los scripts se utiliza para ejecutar los comandos útiles en el desarrollo de la librería

Suggested change

	"test": "echo \"Error: no test specified\" && exit 1"
	"test": "jest"

[reloadercf]

Falta tu repositorio

[reloadercf]

Borra de package.json lo que no utilizas

Suggested change

	"markdown-it": "^11.0.0",
	"marked": "^1.1.0",