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

Skip to content

Traducido archivo library/ctypes.po #397

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 36 commits into from
Aug 22, 2020

Conversation

neosergio
Copy link
Contributor

Closes #322

@neosergio
Copy link
Contributor Author

neosergio commented Jun 14, 2020

Regarding latest Travis-Ci build failed I think I'm dealing with this issue python/python-docs-pt-br#8, so workaround should be the same.

Bug still persist: https://bugs.python.org/issue39229

@humitos
Copy link
Collaborator

humitos commented Jun 14, 2020

Do you know why this is failing in this particular file and not in others where we are already using :ref:evento de auditoría <auditing>? I'm not really sure to understand the problem in the linked issues.

@humitos
Copy link
Collaborator

humitos commented Jun 14, 2020

I think the problem here is that we have this paragraph,

msgid ""
"Raises an :ref:`auditing event <auditing>` ``ctypes.dlopen`` with argument "
"``name``."
msgstr ""
"Lanza un :ref:`evento de auditoría <auditing>` ``ctypes.dlopen`` con el "
"argumento ``name``."

that do not belong to any .rst file (note that they have no header like #: ../Doc/library/ctypes.rst:....

I'm taking a look to see if they are in the original, or we should remove them.

@humitos
Copy link
Collaborator

humitos commented Jun 14, 2020

@neosergio thanks for pointing out to this issue. I'm following them now and I will keep a closer eye on the resolution of them.

For now, I'm marking those translation that are failing as fuzzy (Need Work) so the WARNING issue disappear for now and we can continue moving forward.

These paragraph are failing with

cpython/Doc/library/ctypes.rst:: WARNING: inconsistent term references in
translated message. original: [], translated: [':ref:`evento de auditoría
<auditing>`']

Marking them as fuzzy is a workaround for now, and it will allow us to easily
detect them in the future and fix them.

References:

* https://bugs.python.org/issue39229
* python/python-docs-pt-br#8

There is no way to skip this check only in one paragraph yet:
sphinx-doc/sphinx#3985
@humitos
Copy link
Collaborator

humitos commented Jun 14, 2020

@neosergio I fixed this by marking them as fuzzy for now. Please, do a git pull in your local instance and continue from there.

@neosergio
Copy link
Contributor Author

It seems to be a problem just for this file, I read library/functions.po with similar line and looks everything fine, so I will update my local instance and commit again using that reference. File is almost complete but: auditing event <auditing>

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

ups, nunca envié este review, core que me quedé a la mitad, pero paso los que tenía de momento.

neosergio and others added 10 commits June 29, 2020 11:12
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Collaborator

hay varias palabras que podrías agregar al dict, como ctypes, pero hay otras que parecen venir de problemas con los roles de sphinx, le voy a echar un vistazo al archivo.

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

dll y dlls también se debería agregar IMHO

neosergio and others added 2 commits July 2, 2020 08:17
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
neosergio and others added 8 commits July 2, 2020 08:17
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Collaborator

@neosergio como vamos!? este finde estamos a full! avisanos si necesitas alguna ayuda con este PR

@cmaureir
Copy link
Collaborator

@neosergio si no tienes tiempo para continuar con el PR, avisanos para poder finalizar lo que queda y hacer el merge.

@cmaureir
Copy link
Collaborator

༼ つ ◕_◕ ༽つ
Arreglados los problemas relacionados a la build de este PR.
Se necesitan revisores. ᕕ( ᐛ )ᕗ

@cmaureir cmaureir merged commit a43941e into python:3.8 Aug 22, 2020
@cmaureir
Copy link
Collaborator

Gracias @neosergio por la traducción 🎉

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/ctypes.po
3 participants