Skip to content

Conversation

@alftri
Copy link
Contributor

@alftri alftri commented Dec 16, 2021

Translated the remaining items of library/configparser and added myself to the TRANSLATORS file

Closes #1756

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Muchas gracias por la contribución, ¡está súper buena! Encontré sólo un detallito, después de eso estamos y mergeamos.

msgstr ""
"Los comentarios presentes en el fichero original no se mantienen cuando se "
"vuelve a escribir la configuración. Qué es considerado un comentario depende "
"de los valores asignados a *comment_prefix* y *inline_comment_prefix*."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"de los valores asignados a *comment_prefix* y *inline_comment_prefix*."
"de los valores asignados a *comment_prefix* e *inline_comment_prefix*."

"cadena de caracteres específica (``=`` ó ``:`` por defecto [1]_). De forma "
"predeterminada, los nombres de sección son sensibles a mayúsculas y "
"minúsculas pero las claves no [1]_. Los espacios al inicio y final de las "
"claves y valores son eliminados. Los valores pueden ser omitidos, en cuyo "
Copy link
Collaborator

Choose a reason for hiding this comment

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

Faltó aquí también ajustar la traducción, y de hecho por eso es que falla el test en GitHub. El original dice "Values can be omitted if the parser is configured to allow it [1]_"

@aroquega
Copy link
Collaborator

hola @alftri , en la descripción tienes que poner el "Closes #1756"

@cmaureir cmaureir changed the title Closes #1756 Finalizando library/configparser.po Dec 17, 2021
@alftri
Copy link
Contributor Author

alftri commented Dec 17, 2021

Gracias a todos por la revisión y el feedback. Espero que el nuevo commit esté ok.

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

Gracias @alftri por las revisiones, se ve todo OK 🚀

Como sugerencia, para la próxima puedes ir marcando los reviews en el PR como resueltos, así queda claro qué cosas ya han sido corregidas y cuáles aún falta por ver.

@rtobar rtobar merged commit a246eee into python:3.10 Dec 18, 2021
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/configparser.po'

3 participants