Skip to content

Finalizando library/configparser.po #1771

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 18, 2021
Merged

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.

@@ -1424,6 +1430,9 @@ msgid ""
"the configuration back. What is considered a comment, depends on the given "
"values for *comment_prefix* and *inline_comment_prefix*."
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*."

"Un archivo de configuración consiste de secciones, cada una iniciada por una "
"cabecera ``[section]``, seguida por registros clave-valor, separados por una "
"Un archivo de configuración consta de secciones, cada una iniciada por una "
"cabecera ``[section]``, seguida por registros clave-valor separados por una "
"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