Mconcoba/entry-challenge#8
Conversation
LucasSolares
left a comment
LucasSolares
left a comment
There was a problem hiding this comment.
Buen comienzo, soluciona los comentarios asignados aquí abajo! 😉
| }, | ||
| } | ||
|
|
||
| async function sendMyInformation(){ |
There was a problem hiding this comment.
ES8 incorpora async/await pero una función no debería ser async si no se esta usando la palabra await en ninguna parte en el cuerpo de la misma.
JavaScript async/await Hechale un vistazo a este link para más información. 👀
There was a problem hiding this comment.
@LucasSolares procura utilizar este markdown cuando hablemos de código, como keywords, etc.
| import fs from "fs"; | ||
| import schema from "./schema.json"; | ||
| import server from "./server"; | ||
| import { Request, Response } from "express"; |
There was a problem hiding this comment.
Estos imports no son necesarios en este archivo eliminalos por favor 😁
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@MConcoba vas por buen camino, por favor actualiza con los comentarios de @LucasSolares
Gracias Lucas, buen trabajo en los comments.
| @@ -0,0 +1,1178 @@ | |||
| { | |||
There was a problem hiding this comment.
@MConcoba este file no debería estar en este PR. No hay necesidad de modificar el package.json. Además estamos usando yarn y éste genera yarn.lock.
|
Gracias @LucasSolares y @netpoe por los comments |
| import { request } from "http"; | ||
|
|
||
| import myInformation from "./data"; | ||
| const password = process.env.PASSWORD_SECRET; |
There was a problem hiding this comment.
El uso de variables globales en JS, no es una buena practica lo mejor es declararlas en el scope de la función donde se van a utilizar o en este caso como es una variable de entorno puedes simplemente usarla dentro de tu función donde la necesites. 👌
| /dist | ||
| /data | ||
| src/client.ts No newline at end of file | ||
| /pakage-lock.json No newline at end of file |
There was a problem hiding this comment.
No es necesario agregar el archivo package-lock.json solo borrarlo de tu carpeta ademas solo como dato importante los archivos lock nunca se agregan a un gitignore ya que estos mapean las librearías para que la instalación sea más rapida. 🐱🏍
| }, | ||
| } | ||
|
|
||
| export = myInformation; No newline at end of file |
There was a problem hiding this comment.
Algo que no es obligatorio, pero da buena estética al código es agregar una linea en blanco al final de los archivos de código, te recomiendo que lo hagas!
There was a problem hiding this comment.
@MConcoba puedes instalar la extensión prettier en tu editor.
También puedes agregar esto a tus settings.json:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
| }, | ||
| { | ||
| question: "code is poetry, because:", | ||
| answer: "Poetry is art, art is defined as a way to convey feelings creating something that is unique and original.", |
| }, | ||
| } | ||
|
|
||
| export = myInformation; |
There was a problem hiding this comment.
Este syntax no es muy claro, es mejor si hacemos:
| export = myInformation; | |
| export default myInformation; |
3 participants
No description provided.