Skip to content

Eval: run a code string - #303

Merged
joaquinelio merged 9 commits into
javascript-tutorial:masterfrom
joaquinelio:jeval
Jul 15, 2020
Merged

Eval: run a code string#303
joaquinelio merged 9 commits into
javascript-tutorial:masterfrom
joaquinelio:jeval

Conversation

@joaquinelio

@joaquineliojoaquinelio commented Jul 15, 2020

Copy link
Copy Markdown
Member

No description provided.

@javascript-translate-bot
javascript-translate-bot requested a review from a teamJuly 15, 2020 06:56
@iliakaniliakan mentioned this pull request Jul 15, 2020
Open
maksumi
maksumi approved these changes Jul 15, 2020
View reviewed changes

@maksumimaksumi left a comment

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.

Me agrada, no tiene detalle. La mayoría solo son sugerencias alternativas de trad, sólo eso

Comment thread1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Outdated
Comment thread1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Outdated
Comment thread1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Outdated
Comment thread1-js/99-js-misc/02-eval/article.md Outdated
Comment thread1-js/99-js-misc/02-eval/article.md Outdated
Comment thread1-js/99-js-misc/02-eval/article.md Outdated
Comment thread1-js/99-js-misc/02-eval/article.md Outdated
joaquinelioand others added 4 commits July 15, 2020 13:56
@joaquinelioMaksumi Murakami
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
@joaquinelioMaksumi Murakami
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>
@joaquinelioMaksumi Murakami
Update 1-js/99-js-misc/02-eval/article.md
d3125a1
Co-authored-by: Maksumi Murakami <maksumimurakami@gmail.com>
joaquinelioand others added 2 commits July 15, 2020 14:14
@joaquinelioMaksumi Murakami
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>
@joaquinelioMaksumi Murakami
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>
@joaquinelioMaksumi Murakami
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

joaquinelio commented Jul 15, 2020

Copy link
Copy Markdown
MemberAuthor

@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

maksumi commented Jul 15, 2020•
edited
Loading

Copy link
Copy Markdown
Collaborator

@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

@vplentinaxvplentinax 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.

Ok.!

@joaquinelio
joaquinelio merged commit 80c8685 into javascript-tutorial:masterJul 15, 2020
@javascript-translate-bot

javascript-translate-bot commented Jul 15, 2020

Copy link
Copy Markdown

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

@joaquinelio
joaquinelio deleted the jeval branch July 15, 2020 22:30
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maksumimaksumimaksumi approved these changes
@vplentinaxvplentinaxvplentinax approved these changes

Assignees

No one assigned

Labels

ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@joaquinelio@maksumi@javascript-translate-bot@vplentinax