Sitelet https://web.archive.org/web/20210117183139/https://github.com/python/python-docs-es/pull/397
Skip to content
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

Traducido archivo library/ctypes.po #397

Merged
merged 36 commits into from Aug 22, 2020

Conversation

@neosergio
Copy link
Contributor

@neosergio neosergio commented Jun 13, 2020

Closes #322

@neosergio
Copy link
Contributor Author

@neosergio 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 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 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 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 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

@neosergio neosergio commented Jun 14, 2020

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

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

library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
neosergio and others added 10 commits Jun 29, 2020
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jun 29, 2020

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

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

library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
library/ctypes.po Outdated Show resolved Hide resolved
neosergio and others added 2 commits Jul 2, 2020
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
neosergio and others added 8 commits Jul 2, 2020
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Jul 25, 2020

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

@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Aug 10, 2020

@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

@cmaureir cmaureir commented Aug 15, 2020

༼ つ ◕_◕ ༽つ
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
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Aug 22, 2020

Gracias @neosergio por la traducción 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.