Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Translate c-api/exceptions.po file #1578

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 29, 2021
Merged

Translate c-api/exceptions.po file #1578

merged 2 commits into from
Oct 29, 2021

Conversation

manudefrutosvila
Copy link
Contributor

It closes #1531

@cmaureir
Copy link
Collaborator

falla pospell https://python-docs-es.readthedocs.io/es/3.10/reviewers-guide.html?highlight=powrap#pospell-falla

@manudefrutosvila
Copy link
Contributor Author

manudefrutosvila commented Oct 28, 2021 •
edited
Loading

No se muy bien que debo hacer, leí la docu del enlace, y al final logré lanzar el comando de pospell, y me retorna lo siguiente:

image

primero unas lineas de error de iso a utf-8, es por esto que falla pospell en ci?

y luego retorna muchas palabras... ¿las debo agregar a algún sitio?

@manudefrutosvila manudefrutosvila force-pushed the translate-c-api-exceptions branch 2 times, most recently from 206ce7f to 8fcf1aa Compare October 28, 2021 14:25
manudefrutosvila
manudefrutosvila commented Oct 28, 2021
View reviewed changes
c-api/exceptions.po
@@ -775,6 +772,11 @@ msgid ""
"if so, invokes the corresponding signal handler. If the :mod:`signal` "
"module is supported, this can invoke a signal handler written in Python."
msgstr ""
"Si la función se llama desde el hilo principal y bajo el intérprete "
"principal de Python, verifica si se ha enviado una señal a los procesos y, "
"de ser así, invoca el manejador de señales correspondiente. Si el módulo :"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

he probado juntar

:mod:`signal`

pero powrap se queja y lo separa

Copy link
Collaborator

@cmaureir cmaureir Oct 28, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

no hay ningún problema con eso, al igual que en python:

a = ("ho"
    "la")
# será "hola"

cmaureir
cmaureir requested changes Oct 28, 2021
View reviewed changes
@cmaureir
Copy link
Collaborator

No se muy bien que debo hacer, leí la docu del enlace, y al final logré lanzar el comando de pospell, y me retorna lo siguiente:
...

primero unas lineas de error de iso a utf-8, es por esto que falla pospell en ci?

Puedes ver el error de pospell, haciendo click en 'Details' y mirando el mensaje.

y luego retorna muchas palabras... ¿las debo agregar a algún sitio?

Esas palabras, es porque parece que no tienes un diccionario español instalado con hunspell.
Puedes usar otras herramientas, como 'aspell' ?

@manudefrutosvila manudefrutosvila force-pushed the translate-c-api-exceptions branch from 8fcf1aa to ab943d7 Compare October 28, 2021 21:46
@manudefrutosvila
Copy link
Contributor Author

manudefrutosvila commented Oct 28, 2021 •
edited
Loading

Puedes ver el error de pospell, haciendo click en 'Details' y mirando el mensaje.

Donde te refieres hacer click en 'Details' ? uso pospell desde la terminal, no veo nada más que eso

Esas palabras, es porque parece que no tienes un diccionario español instalado con hunspell. Puedes usar otras herramientas, como 'aspell' ?

instalé hunspell también y agregué los diccionarios es_ES y es_AR de libreoffice
intentaré probar con aspell

Gracias!!

@manudefrutosvila
Copy link
Contributor Author

Ahh 'Details' te refieres para ver en la CI, no?
Si eso lo miré, pero no identifiqué ningún error en concreto de pospell 😩

@manudefrutosvila
Copy link
Contributor Author 

cpython/Doc/c-api/exceptions.rst:553: WARNING: inconsistent term references in translated message. original: [':c:func:`PyErr_CheckSignals`'], translated: [':const:`SIGINT`', ':c:func:`PyErr_CheckSignals`', ':const:`SIGINT`']
cpython/Doc/c-api/exceptions.rst:562: WARNING: inconsistent term references in translated message. original: [':data:`signal.SIG_DFL`', ':data:`signal.SIG_IGN`'], translated: [':const:`SIGINT`', ':data:`signal.SIG_DFL`', ':data:`signal.SIG_IGN`']

En Poedit me salieron unas marcas de textos ya traducidos como "Pendientes de revisión" que creo que las marqué como revisadas, y creo que por eso hay textos que no he traducido yo que sale el warning. Voy a revisarlos

@manudefrutosvila manudefrutosvila force-pushed the translate-c-api-exceptions branch from 93c6efa to ea22f91 Compare October 28, 2021 22:31
@manudefrutosvila manudefrutosvila force-pushed the translate-c-api-exceptions branch from ea22f91 to b75e523 Compare October 28, 2021 22:37
@manudefrutosvila
Copy link
Contributor Author

Ahora parece que sí 🎉

@cmaureir cmaureir merged commit b016e94 into python:3.10 Oct 29, 2021
@cmaureir
Copy link
Collaborator

Muchas gracias @manudefrutosvila

cmaureir pushed a commit that referenced this pull request Dec 7, 2021
@manudefrutosvila @cmaureir
Translate c-api/exceptions.po file (#1578)
0bf107e 
* Translate c-api/exceptions.po file

* Fix the messages marked as fuzzy in the c-api/exceptions.po file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmaureir cmaureir cmaureir approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Translate 'c-api/exceptions.po'
2 participants
@manudefrutosvila @cmaureir