Skip to content

Traducido library/ast#484

Merged
cmaureir merged 5 commits into
python:3.8from
cmaureir:traduccion-library-ast
Jul 26, 2020
Merged

Traducido library/ast#484
cmaureir merged 5 commits into
python:3.8from
cmaureir:traduccion-library-ast

Conversation

@cmaureir

Copy link
Copy Markdown
Collaborator

Closes #469

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

¡Muchas gracias @cmaureir!

Dejé algunas sugerencias y con eso, creo quedaría listo el archivo :)

Comment thread library/ast.po Outdated
Comment thread library/ast.po
Comment on lines +144 to +145
"`end_col_offset` son las correspondientes compensaciones de bytes UTF-8 del "
"primer y último token que generó el nodo. El desplazamiento UTF-8 se "

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.

Suggested change
"`end_col_offset` son las correspondientes compensaciones de bytes UTF-8 del "
"primer y último token que generó el nodo. El desplazamiento UTF-8 se "
"`end_col_offset` son las correspondientes compensaciones de bytes UTF-8 del "
"primer y último token que generó el nodo. El desplazamiento UTF-8 se "

Acá usas "compensaciones" y "desplazamiento" para "offsets", te parece si lo estandarizamos por alguno que encaje en ambas líneas?

Comment thread library/ast.po
"If there are keyword arguments, they will set the attributes of the same "
"names to the given values."
msgstr ""
"Si hay argumentos de palabras clave, establecerán los atributos de los "

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.

Suggested change
"Si hay argumentos de palabras clave, establecerán los atributos de los "
"Si hay argumentos por palabras clave, establecerán los atributos de los "

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

al final decidimos aceptar los dos :P "por" y "de"

Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated
Comment thread library/ast.po Outdated

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

Encontré una última cosita :)

Comment thread library/ast.po Outdated
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
@narvmtz

narvmtz commented Jul 25, 2020

Copy link
Copy Markdown
Contributor

Solo faltaría hacerle powrap y creo que con eso pasaría Travis :)

@cmaureir cmaureir requested a review from narvmtz July 26, 2020 07:50
@cmaureir

Copy link
Copy Markdown
Collaborator Author

@narvmtz cuando le des approve mergeamos este :)

@cmaureir cmaureir merged commit 80bbf94 into python:3.8 Jul 26, 2020
@cmaureir cmaureir deleted the traduccion-library-ast branch July 26, 2020 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Translate library/ast.po

2 participants