Conversation
Israel-Laguan
requested review from
anaceciliamolinares,
angelik0828 and
samuel-vergara-leon
Co-authored-by: Israel Antonio Rosales Laguan <israellaguan@gmail.com>
Co-authored-by: Israel Antonio Rosales Laguan <israellaguan@gmail.com>
Co-authored-by: Israel Antonio Rosales Laguan <israellaguan@gmail.com>
Co-authored-by: Víctor Peña <66505715+Katsu08@users.noreply.github.com>
…ordev/backend-template into feat/add-translations-of-docs
Co-authored-by: Víctor Peña <66505715+Katsu08@users.noreply.github.com>
Co-authored-by: Víctor Peña <66505715+Katsu08@users.noreply.github.com>
heliomar-pena
left a comment
heliomar-pena
left a comment
There was a problem hiding this comment.
Thanks for the changes! I have noticed some little things and I left a comment in this review
| ## Tabulaciones y sangría | ||
|
|
||
| * No use el carácter de tabulación, use espacios en su lugar | ||
| * El tamaño de la pestaña es de 2 espacios |
There was a problem hiding this comment.
"El tamaño de cada tabulación" o "cada sangría" creo que se entiende más que espacio de cada pestaña
|
|
||
| * No use el carácter de tabulación, use espacios en su lugar | ||
| * El tamaño de la pestaña es de 2 espacios | ||
| * La sangría es de 1 tamaño de pestaña (2 espacios) |
There was a problem hiding this comment.
Siento que esto es confuso. Entonces la sangría y la pestaña son lo mismo, por qué diferenciarlo?
| * No use el carácter de tabulación, use espacios en su lugar | ||
| * El tamaño de la pestaña es de 2 espacios | ||
| * La sangría es de 1 tamaño de pestaña (2 espacios) | ||
| * La sangría de continuación es de 1 tabulación |
There was a problem hiding this comment.
Ya aquí me perdí, no estoy seguro de qué es una tabulación, qué es un espacio y qué es una sangría.
Atrás creo que se mencionó que no usaremos tabulaciones?
| * Las líneas vacías no deben llevar sangrías | ||
| * Se pueden agregar líneas en blanco para facilitar la lectura | ||
| * Se permiten 2 líneas en blanco consecutivas | ||
| * Las ramas `case` en las sentencias switch deben estar sangradas |
There was a problem hiding this comment.
En mi caso no conozco el término "sangrado" en este contexto, y buscarlo en google se me dificultaría. Existe alguna otra forma de expresarlo?
| ++ const foo = 'just a string'; | ||
| -- const bar = "I'm using single quotes inside"; |
There was a problem hiding this comment.
No me queda claro por qué uno tiene "++" y otro "--", por lo que yo entiendo eso podría significar que se añadió una línea y se borró la otra, pero creo que no es la intención. Pueden validar?
There was a problem hiding this comment.
Si, creo que es como decir "esto esta bien" (++) contra "esto esta mal, borralo" (--)
| async function foo() { | ||
| const bar = await fetchBar(); | ||
|
|
||
| if (!!bar) { | ||
| doSomething(); | ||
| } | ||
| } | ||
|
|
||
| // Pero esto no | ||
| async function foo() { | ||
| const bar = await fetchBar(); | ||
|
|
||
| if (!!bar) doSomething(); | ||
| } |
There was a problem hiding this comment.
Add code block
//example4 participants
No description provided.