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

Eval: run a code string by joaquinelio · Pull Request #303 · javascript-tutorial/es.javascript.info · GitHub

/ es.javascript.info Public

Eval: run a code string - #303

Merged
joaquinelio merged 9 commits into
javascript-tutorial:masterfrom
joaquinelio:jeval
Jul 15, 2020
ConversationCommits (9)ChecksFiles changed
Merged

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

Conversation

joaquinelio commented Jul 15, 2020

Copy link
Copy Markdown
Member

No description provided.

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

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.

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

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 Outdated
Comment thread 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Outdated
Comment thread 1-js/99-js-misc/02-eval/1-eval-calculator/solution.md Outdated
Comment thread 1-js/99-js-misc/02-eval/article.md Outdated
Comment thread 1-js/99-js-misc/02-eval/article.md Outdated
Comment thread 1-js/99-js-misc/02-eval/article.md Outdated
Comment thread 1-js/99-js-misc/02-eval/article.md Outdated
joaquinelio and others added 4 commits July 15, 2020 13:56
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>
Update solution.md
dc392a0
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>
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
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>
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>
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 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 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 self-requested a review July 15, 2020 21:07
vplentinax approved these changes Jul 15, 2020
View reviewed changes

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.!

joaquinelio merged commit 80c8685 into javascript-tutorial:master Jul 15, 2020

javascript-translate-bot commented Jul 15, 2020

Copy link
Copy Markdown

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

joaquinelio deleted the jeval branch July 15, 2020 22:30
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

Back | FazBrowse Home | New Git URL