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

@joaquinelio joaquinelio commented Jul 15, 2020

Copy link
Copy Markdown
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

@maksumi maksumi 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 thread 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Resolved
Comment thread 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Resolved
Comment thread 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Resolved
Comment thread 1-js/99-js-misc/02-eval/article.md Resolved
Comment thread 1-js/99-js-misc/02-eval/article.md Resolved
Comment thread 1-js/99-js-misc/02-eval/article.md Resolved
Comment thread 1-js/99-js-misc/02-eval/article.md Resolved
joaquinelio and others added 4 commits July 15, 2020 13:56
@joaquinelio Maksumi 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
@joaquinelio Maksumi 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>
@joaquinelio Maksumi Murakami
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 Maksumi 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>
@joaquinelio Maksumi 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>
@joaquinelio Maksumi 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
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

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

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

Ok.!

@joaquinelio
joaquinelio merged commit 80c8685 into javascript-tutorial:master Jul 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maksumi maksumi maksumi approved these changes
@vplentinax vplentinax vplentinax 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