Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
cmaureir merged 10 commits intopython:3.12frommmmarcos:fix-more-fuzzy-entries
Nov 18, 2024

Conversation

mmmarcos
Copy link
Collaborator

Fuzzy entries everywhere!

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedNov 6, 2023
edited
Loading

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 reacted with thumbs up emoji

@mmmarcos
Copy link
CollaboratorAuthor

@rtobar por qué hay que hacer un merge de3.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 de3.12, no sería mejor hacer un rebase.

@rtobar
Copy link
Collaborator

@rtobar por qué hay que hacer un merge de3.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.

mmmarcos reacted with thumbs up emoji

Copy link
Collaborator

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

@cmaureir
Copy link
Collaborator

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
CollaboratorAuthor

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 archivosson 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.

cmaureir reacted with thumbs up emoji

Copy link
Collaborator

@cmaureircmaureir left a comment

Choose a reason for hiding this comment

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

mejor mergeamos esto antes de la migración

mmmarcos reacted with thumbs up emoji
@cmaureircmaureir merged commitb057b69 intopython:3.12Nov 18, 2024
4 checks passed
@mmmarcosmmmarcos deleted the fix-more-fuzzy-entries branchNovember 30, 2024 16:26
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@rtobarrtobarrtobar left review comments

@cmaureircmaureircmaureir 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.

3 participants
@mmmarcos@cmaureir@rtobar

[8]ページ先頭

©2009-2025 Movatter.jp