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

Fix (many) more fuzzy entries in the library directory #2736

Open
wants to merge 7 commits into
base: 3.12
Choose a base branch
from

Conversation

mmmarcos
Copy link
Collaborator

@mmmarcos mmmarcos commented Nov 6, 2023

Fuzzy entries everywhere!

Copy link

github-actions bot commented Nov 6, 2023

All entries translated, horray! 🎉

@cmaureir
Copy link
Collaborator

Hay un par de problemas aún con sphinx-lint

library/email.generator.po library/email.headerregistry.po library/email.message.po library/email.mime.po library/filecmp.po library/graphlib.po library/hmac.po library/html.entities.po library/json.po library/keyword.po library/lzma.po library/mailbox.po library/operator.po library/queue.po library/secrets.po library/security_warnings.po library/selectors.po library/stat.po library/string.po library/tomllib.po
library/string.po:426: role missing closing backtick: ':meth:`~object.format, la lógica normal de formateo es evitada.\n' (missing-backtick-after-role)

@mmmarcos
Copy link
Collaborator Author

@rtobar por qué hay que hacer un merge de 3.12 hacia la rama? No sé si está relacionado pero vi otros PR (#2748 (comment)) que se encontraron con el mismo "problema".

De ser necesarios los últimos cambios de 3.12, no sería mejor hacer un rebase.

@rtobar
Copy link
Collaborator

rtobar commented Nov 20, 2023

@rtobar por qué hay que hacer un merge de 3.12 hacia la rama? No sé si está relacionado pero vi otros PR (#2748 (comment)) que se encontraron con el mismo "problema".

No hay que hacer un merge. Lo que pasa es que, por alguna razón que no he tenido el tiempo de averiguar en detalle, algunas veces la GitHub Action que deja el comentario se marea la primera vez que es lanzada. Al hacer más commits sobre la rama que se quiere mergear (ya sea haciendo un merge de 3.12 hacia la rama, o poniendo más commits normales) el bot ya hace bien su trabajo.

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.

Revisado hasta library/email.headerregistry.po

library/__main__.po Show resolved Hide resolved
library/__main__.po Show resolved Hide resolved
library/codeop.po Outdated Show resolved Hide resolved
library/codeop.po Outdated Show resolved Hide resolved
library/codeop.po Outdated Show resolved Hide resolved
library/email.headerregistry.po Outdated Show resolved Hide resolved
@cmaureir
Copy link
Collaborator

cmaureir commented Jan 5, 2024

Tenemos que tener cuidado con este tipo de cambios masivos, principalmente porque si alguien toma un archivo de estos para hacer la traducción, va a tener conflictos si esto es mergeado justo antes del otro.

@mmmarcos
Copy link
Collaborator Author

mmmarcos commented Jan 5, 2024

Tenemos que tener cuidado con este tipo de cambios masivos, principalmente porque si alguien toma un archivo de estos para hacer la traducción, va a tener conflictos si esto es mergeado justo antes del otro.

Me extraña araña! ^^ (conocen ese dicho fuera de Argentina?)

@cmaureir en realidad, sólo modifiqué archivos con entradas fuzzy (sin ninguna entrada para traducir). Estos archivos son listados en el progress report pero no son tenidos en cuenta por el script que crea los issues. Igual me había tomado el trabajo de verificar uno por uno los archivos (puede fallar!) para ver que no haya ningún issue abierto para reducir el riesgo de conflicto.

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.

None yet

3 participants