Skip to content

Eval: run a code string #303

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Jul 15, 2020
Merged

Eval: run a code string #303

merged 9 commits into from
Jul 15, 2020

Conversation

joaquinelio
Copy link
Member

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team July 15, 2020 06:56
@iliakan iliakan mentioned this pull request Jul 15, 2020
Open
maksumi
maksumi approved these changes Jul 15, 2020
View reviewed changes
Copy link
Collaborator

@maksumi maksumi left a comment

Choose a reason for hiding this comment

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

Me agrada, no tiene detalle. La mayoría solo son sugerencias alternativas de trad, sólo eso
joaquinelio and others added 4 commits July 15, 2020 13:56
@joaquinelio
Update 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md
c4ef465 
UH!!
Estoy desolado.
Pero gracias.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update solution.md
dc392a0
@joaquinelio
Update 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md
0fc1688 
No, no más claridad, más corrección. Operando es el miembro, operador es el signo.
Good catch diria un yanqui.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/99-js-misc/02-eval/article.md
d3125a1 
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
joaquinelio and others added 2 commits July 15, 2020 14:14
@joaquinelio
Update 1-js/99-js-misc/02-eval/article.md
f9641fa 
Jugué un poco, me pareció diabolico mas que "el diablo"
Malvado me gusta mas ahora,,,
estuve tentado a poner "malito" tambien, una chiquilinada.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/99-js-misc/02-eval/article.md
cfc3adf 
el plural me da igual, 
pero la fluidez mejoro mucho

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Update 1-js/99-js-misc/02-eval/article.md
9f5e957 
UUUHHH
El peligro de empezar a asemejarsse al inglés, 
la falta de conectores.

Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
@joaquinelio
Copy link
Member Author

@maksumi , me gustó tu revissión, todo tenía sesntido. (tambien las comenté)
Lástima ya no quedan review de las 80 con las que empezamos...

Pero esa es la buena noticia, estamos al día y podemos seguir traduciendo.
Array methods es largo y con varios archivos, quizas siga con ese
¿o lo querés?
Tambien puedo limpiar algun otro de 2019...
avisa

hoy aporto poco, mejor hago bigint aunq ess un poco irrelevante
@maksumi
Copy link
Collaborator

maksumi commented Jul 15, 2020 •
edited
Loading

@maksumi , me gustó tu revissión, todo tenía sesntido. (tambien las comenté)
Lástima ya no quedan review de las 80 con las que empezamos...

Pero esa es la buena noticia, estamos al día y podemos seguir traduciendo.
Array methods es largo y con varios archivos, quizas siga con ese
¿o lo querés?
Tambien puedo limpiar algun otro de 2019...
avisa

hoy aporto poco, mejor hago bigint aunq ess un poco irrelevante

Me hubiera gustado aportar por ello, la timidez gana cuando uno es nuevo y tiene miedo de involucrarse más a fondo por no saber cómo hacerlo o por romper algo en el camino.

Ahorita estoy haciendo ya una traducción pero tal vez hoy quede lista. Como tú consideres necesario, si gustas tómala.

Pd: ¿Aún me cuesta entender el flujo de trabajo de esto o es que olvidaste poner el /done @joaquinelio
@vplentinax vplentinax self-requested a review July 15, 2020 21:07
vplentinax
vplentinax approved these changes Jul 15, 2020
View reviewed changes
Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Ok.!
@joaquinelio joaquinelio merged commit 80c8685 into javascript-tutorial:master Jul 15, 2020
@javascript-translate-bot
Copy link

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉
@joaquinelio joaquinelio deleted the jeval branch July 15, 2020 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge
4 participants
@joaquinelio @maksumi @javascript-translate-bot @vplentinax