Add new files#46
Conversation
reloadercf
left a comment
reloadercf
left a comment
There was a problem hiding this comment.
Yaz me gusta que tu librería resuelve el reto, tengo algunas recomendaciones, la primera es recordarte que esta librería la vas a integrar a tu portafolio de trabajo y debes integrar ayuda o documentación sobre la instalación y lógica que maneja tu librería, eso lo puedes agregar en tu README, otra recomendación es que aproveches las funcionalidades de GitHub, me percate que solo usaste 2 commits, te estas perdiendo de explotar al máximo GitHub, y la ultima recomendación es sobre los test, recuerda puedes usar los test para saber si tu librería trabaja bien a largo plazo, pero si te das cuenta saber si una función es una función no ayuda a ese objetivo de saber si tu librería sigue funcionando a largo plazo, puedes complementar eso con datos reales. Algo que quiero destacar es el aprendizaje del uso de funciones, eso se nota que lo tienes bien dominado.
| @@ -0,0 +1 @@ | |||
| node_modules | |||
There was a problem hiding this comment.
que cool que excluyas node_modules/ seria bueno que tus ejemplos de md pudiesen vivir dentro de una carpeta assets y a su vez ser excluidos.
| { | ||
| "name": "cdmx009-mdlinks", | ||
| "version": "1.0.0", | ||
| "description": "\"Validate MarkDown Links\"", |
There was a problem hiding this comment.
tu descripción debe contener más información de lo que hace tu librería.
| "description": "\"Validate MarkDown Links\"", | |
| "description": "\"Validate MarkDown Links and generate stats\"", |
| @@ -0,0 +1,102 @@ | |||
| POKENDIA GO! | |||
There was a problem hiding this comment.
este archivo debe ir en una carpeta llamada assets y a su vez la carpeta debe ser excluida.
| @@ -0,0 +1,37 @@ | |||
| Data Lovers, viaje extendido | |||
There was a problem hiding this comment.
este archivo debe estar en assets y a su vez excluido.
| const mdLinks = require('./md.js') | ||
| const fetch = require('node-fetch'); | ||
| const chalk = require('chalk'); | ||
| const pathDoc = process.argv[2]; | ||
| let options = { | ||
| validate: false, | ||
| stats: false | ||
| } |
There was a problem hiding this comment.
es bueno ver esta declaración de variables usando const para variables las cuales el valor no muta y usando let para cuando el valor muta.
|
|
||
| const arrayLinksMd = []; | ||
|
|
||
| for (let i in linksMatchInMd) { |
There was a problem hiding this comment.
esta raro el for, solo quiero recordarte su estructura for(let i=algo, i<=n, i++) lo conforma la asignación, después la evaluación y por ultimo la reacción en caso de que se cumpla.
| for (let i in linksMatchInMd) { | |
| for (let i in linksMatchInMd) { |
| arrayLinksMd.push({ | ||
| href: urlsLinksMd.substring(1, urlsLinksMd.length - 1), | ||
| text: textLinksMd.substring(1, textLinksMd.length - 1).slice(0,49), | ||
| file: pathDocument | ||
| }); |
There was a problem hiding this comment.
Tal vez solo con el link y el text es suficiente
| arrayLinksMd.push({ | |
| href: urlsLinksMd.substring(1, urlsLinksMd.length - 1), | |
| text: textLinksMd.substring(1, textLinksMd.length - 1).slice(0,49), | |
| file: pathDocument | |
| }); | |
| arrayLinksMd.push({ | |
| href: urlsLinksMd.substring(1, urlsLinksMd.length - 1), | |
| text: textLinksMd.substring(1, textLinksMd.length - 1).slice(0,49) | |
| }); |
| } | ||
| resolve(arrayLinksMd); | ||
| } else { | ||
| console.log('No es un archivo .md...Ingrese de nuevo'); |
There was a problem hiding this comment.
En ingles
| console.log('No es un archivo .md...Ingrese de nuevo'); | |
| console.log('Not an .md file ... Please login again'); |
|
|
||
| describe('mdLinks', () => { | ||
|
|
||
| it('Debería ser una función', () => { |
There was a problem hiding this comment.
En ingles
| it('Debería ser una función', () => { | |
| it('It should be a function', () => { |
| }); | ||
|
|
||
| }); | ||
|
|
There was a problem hiding this comment.
Considero que los test los puedes complementar implementando tus funciones y con valores reales y esperando respuestas concretas como por ejemplo: pasarle el link let arrayLink=['https://google.com'] y esperando la respuesta "status=200 https://google.com ok"
2 participants
Validación de links totales y únicos