Skip to content
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

ArrayBuffer, binary arrays #532

Merged

Conversation

jespadas
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 24, 2021

CLA assistant check
All committers have signed the CLA.

@joaquinelio
Copy link
Member

joaquinelio commented Jul 24, 2021

hola @jespadas
Todavia no hago la rev, pero anoto:

  • Revisa si tiene "tasks", cada tarea tiene su subdirectorio (aca parece solo una linea, mas rapido para mi si la edito en github)
  • Es importante respetar el estilo y los tiempos, pero la gramatica tiene prioridad. Mmm... mentira. La fluidez es mas importante.
  • Respetar los numeros de linea, es importante cuando hacemos MERGE desde el repo ingles. (no te preocupes en este, lo corrijo facil en la review )
  • el "--" es metadata para "—" significado parecido a ":" que en español no se usa igual. Lo dejo solo en listados.

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No la terminé,
¿podrias corregir la numeracion de lineas? Me es dificil corregirlo dentro de github

No es solo el numero de linea que me sería facil corregir, tambien
aparecio un 4 de la nada (114)
hay metadata separada (deja de funcionar -no puedo recordar donde sorry)
y lineas partidas no solo formato sino el sentido, p ej 1 linea (118) se transformo en 3 (124 125 126) y se confunde con el codigo

no aflojes plis esto estuvo lonely los ultimos meses =)

4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@jespadas
Copy link
Contributor Author

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sigue el problema de la numeracion, tu articulo es casi 10 lineas mas largo. Si dejara as´i el merge desde el ingles saldria siempre mal.

No se cual es el "err de sintaxis" original, plis no confundas con el markup dentro del codigo. No uses correctores de estilo, rompen la estructura.

Me muerdo la lengua con la recomendacion de quitar comas...
hay casos en que la estructura lo exige, ahora pecamos de falta y la gramatica queda mal.
La coma no marca pausas sino estructura. Pero ante la duda prefiero la fluidez (lo correcto seria rescribir la linea pero no vienen de la RAE a auditarnos)

Leiste el coment de la numeracion de lineas? Por ej la linea 118 que separaste en 3 quedó sin sentido.
Para corrregirlo tendraas que comparar linea x linea con el ingles. Si tu editor lo permite, lo recomiendo. En vscode usaba extension "gitlens" pero le pusieron mejoras y se volvio engorroso, a veces prefiero usar 2 ventanas separadas.

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@jespadas
Copy link
Contributor Author

He corregido el número de líneas, regrese al estado original el error que a mi parecer lo tiene el repositorio original. Por lo de las comas, debemos ponernos de acuerdo porque yo lo deje con ciertas pausas ya que para mi la fluides implica otra forma de la que me pides.

@jespadas
Copy link
Contributor Author

/done

@joaquinelio
Copy link
Member

Acabo de ver lo que creo que ves como error. Pero lo tengo que entender mejor.
Si es error, mandamos el PR al ingles.
Por norma lo corrijo primero en inglés. A veces rechaza el cambio o lo arregla de otra forma.

Comas: Poné todas las que quieras que no te reprimo más, en cambio solo anoto:
*A veces 2 comas actuan como un separador que se lee como si estuviera entre parentesis, eso esta mal cuando la frase pierde coherencia. Acá es donde me quejo (ej linea 7, creo que yerra mucho asi en ingles pero un corrector nativo me lo negó. Quizas confundo los casos).
*No siempre se puede quitar o reemplazar con "y". No conviene si la idea importante se confunde y esto es importante en un tutorial. No sigo reglas gramaticales, veo qué quiere comunicar y si se lee cómodo (la gramatica ayuda).
Si queres devolver las comas, no problem. Sorry por el incordio.

Me pongo a revisarlo.

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no veo que haya err de sintaxis,
online se ven y funcionan

4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
4-binary/01-arraybuffer-binary-arrays/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

jespadas and others added 3 commits July 28, 2021 17:16
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@jespadas
Copy link
Contributor Author

/done

@joaquinelio joaquinelio merged commit 37b366c into javascript-tutorial:master Jul 28, 2021
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

@joaquinelio
Copy link
Member

biutiful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants