FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

The old "var" by joaquinelio · Pull Request #312 · javascript-tutorial/es.javascript.info · GitHub

The old "var" - #312

Merged
joaquinelio merged 29 commits into
javascript-tutorial:masterfrom
joaquinelio:oldvar
Jul 25, 2020
Merged

The old "var"#312
joaquinelio merged 29 commits into
javascript-tutorial:masterfrom
joaquinelio:oldvar

Conversation

Copy link
Copy Markdown
Member

Basado en el repo EN,
varios cambios respecto al nuestro.

vplentinax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Lo encontre algo confuso. También hay líneas que no coinciden. Me gustaria que arreglaras eso y así volver a revisarlo. También agregué algunas sugerencias de cambios que ví por encima.

Copy link
Copy Markdown

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

vplentinax commented Jul 25, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

Basado en el repo EN,
varios cambios respecto al nuestro.

No había visto los cambios, tenía previsto hacer la revisión de sincronización el domingo, pero ya añadí lo nuevo ahora confiando que se arreglarían los problemas de las líneas que no coincidían, pero veo que no sucedió, se creó un pequeño conflicto.

Editado:
veo mejor que cierres y vuelvas a abrir el PR a ver si coincide con los nuevos cambios y se hace más fácil la revisión.
¿Tú que dices?

joaquinelio and others added 13 commits July 24, 2020 23:31
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Tuve que penarlo un poco...
No está nada mal "alcance"

Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
De acuerdo, no me gusto cdo lo escribi.  No es lo que dice el original sino mejor.
Tamb voy a cambiar "mayoria" suena ffeo

Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
mas animal que bestia, "soy animal de ciudad"..
me da igual, hecho.

Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>

joaquinelio commented Jul 25, 2020
edited
Loading

Copy link
Copy Markdown
Member Author

@vplentinax
ah, primero hice loas sug
desp resolve
desp resolvi confl y quedo... no se, una vez q se mete el P..O >>>===<<< es imposible confiar en las lineas
desp vi tu ssug de cerrarlo.. no need, conflicto siimple
EDIT:
Al resolver conflicto, el diff se actualizó solo, asi que la revision sera la correcta.
Algunos sug aplicados, otros loss reescribí
*me suena comodo viejo nuevo
apliqué sug porque supongo que muchos le da igual
y algunos como vos les choque, so, mejor.
*Otros hacen a la fluidez de lectura, muy buenas sugerencias ok
*reformulé lin 31, me aparté del literal espero sea más claro...
*ah, acá usamos saltear (8)... JA JA!! RAE saltear (me gusta 2)
1.Sofreír
2.Salir a los caminos y robar a los pasajeros.
...
8 .Omitir ¿8? ¡BBBUUUUU! (¿puedo mandar pr a la rae? Ilya rechazo dos mios asi q baja autoestima)
Pero tambien entendemos saltar, asi que ok.

Copy link
Copy Markdown
Member Author

/done

maksumi commented Jul 25, 2020

Copy link
Copy Markdown
Collaborator

Aguanten, ya va mi review

vplentinax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

OK, se solucionó el problema de líneas. Se lee más fluido. Correcto!

maksumi left a comment
edited
Loading

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Muy buena traducción, algunos typos aún y esos mosquitos "el" en la línea 4 y 5 donde viviriamos más tranquilos si no estuvieran sin importar si fuera "código viejo" ó "'' antiguo"...
Arregla eso y te doy 5 estrellas....
Perdón, perdón. Solo esos detalles y... ¡Ya todo chido pa'l merge!

joaquinelio and others added 9 commits July 25, 2020 03:20
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>

joaquinelio commented Jul 25, 2020
edited
Loading

Copy link
Copy Markdown
Member Author

@maksumi
FAAAA.., que buena vista, acepte todas lass ssugg

pero
"mosquitos" creo que es mejor "el codigo", pero las acepte porque queria las 5 estrellas
jejej

enteder el codigo
entender codigo
entiendo que te parecio incomodo, depende de cuan especifico sea

"entender sánscrito"
"entender el código que hice ayer"
nuestro caso es bien ambiguo asi que creo qye van las dos formas ok

ah, preaceptado? merde!!! digo MERGE!!!!!

joaquinelio merged commit d7bc432 into javascript-tutorial:master Jul 25, 2020

Copy link
Copy Markdown

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

joaquinelio deleted the oldvar branch July 25, 2020 06:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL