10000 dummy; by rtobar · Pull Request #3 · rtobar/python-docs-es · GitHub
[go: up one dir, main page]

Skip to content

dummy; #3

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

Closed
wants to merge 6 commits into from
Closed

dummy; #3

wants to merge 6 commits into from

Conversation

rtobar
Copy link
Owner
@rtobar rtobar commented Nov 1, 2023

No description provided.

El script está diseñado para ser usado ya sea como un hook de
pre-commit, o (principalmente) dentro de una nuevo paso del GitHub
action que revisa PRs, de tal modo que se añada y actualice
automáticamente un comentario en el PR indicando si todas las entradas
del archivo siendo traducido han sido completadas o no.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
@rtobar rtobar force-pushed the test-pr-comments branch 4 times, most recently from a1362cf to c73e835 Compare November 1, 2023 12:52
Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147
fuzzy library/filecmp.po 200

1 similar comment
Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147
fuzzy library/filecmp.po 200

Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147
fuzzy library/filecmp.po 200

2 similar comments
Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147
fuzzy library/filecmp.po 200

Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147
fuzzy library/filecmp.po 200

Copy link
github-actions bot commented Nov 1, 2023

Entries missing translation, details follow:

Reason File Line
fuzzy library/filecmp.po 147

Copy link
github-actions bot commented Nov 1, 2023

All entries translated, horray! 🎉

rtobar added a commit to python/python-docs-es that referenced this pull request Nov 6, 2023
Este PR agrega una nueva acción que agrega un comentario a los PRs que
editan archivos .po indicando qué entradas aún no tienen una traducción.

Un ejemplo de esto funcionando se puede ver en
rtobar#3. Hay varios comentarios
automáticos porque use ese PR para debuguear el proceso, pero la idea es
que finalmente haya un solo comentario que se va actualizando (como se
puede ver en el último comentario, que está editado).

Otro ejemplo se puede ver en
rtobar#5, donde el PR viene desde
un fork en vez de venir desde el mismo repositorio.

Otra opción habría sido agregar un check a CI que falle si faltan
entradas, pero en 3.11 ya tuvimos el caso en que *tuvimos* que dejar
entradas fuzzy a propósito.

---------

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
@rtobar rtobar closed this Nov 20, 2023
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.

1 participant
0