-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/add translations of docs #30
base: dev
Are you sure you want to change the base?
Conversation
Co-authored-by: Israel Antonio Rosales Laguan <[email protected]>
Co-authored-by: Israel Antonio Rosales Laguan <[email protected]>
Co-authored-by: Israel Antonio Rosales Laguan <[email protected]>
Co-authored-by: Víctor Peña <[email protected]>
…ordev/backend-template into feat/add-translations-of-docs
Co-authored-by: Víctor Peña <[email protected]>
Co-authored-by: Víctor Peña <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add code block
//example
No description provided.