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

Traducido archivo library/decimal#2752

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
rtobar merged 8 commits intopython:3.12fromjcaalzago:decimal.po
Nov 25, 2023
Merged

Conversation

jcaalzago
Copy link
Contributor

@jcaalzagojcaalzago commentedNov 22, 2023
edited
Loading

Closes#2550

@github-actionsGitHub Actions
Copy link

All entries translated, horray! 🎉

@rtobar
Copy link
Collaborator

@jcaalzago puedes actualizar la descripción del PR para que diga "Closes #xxxx" ahí y no en el título? De lo contrario no se enlaza automáticamente este PR con el issue correspondiente. Gracias!

@jcaalzago
Copy link
ContributorAuthor

¿Estaría ya correcto?

@jcaalzagojcaalzago changed the titleTraducido archivo library/decimal Closes #2550Traducido archivo library/decimalNov 22, 2023
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.

@jcaalzago tremendo trabajo! Estaba todo súper, súper bien, felicitaciones por tu primera contribución! Te dejo un par de comentarios muy pequeños; después de que los resuelvas ya hacemos el merge.

@jcaalzago
Copy link
ContributorAuthor

jcaalzago commentedNov 23, 2023 via email
edited
Loading

Muchas gracias,probablemente falte hacer el powrap, me dio un error de rewarp alsubirlo... y he subido otra revisión antes de ver este comentario.... meimagino que no pasará el test...Si no lo pasa de nuevo, lo hago y lo subo.Muchas gracias por todas las indicaciones :)SaludosEl jue, 23 nov 2023 a las 16:25, rtobar ***@***.***>)escribió:
***@***.**** commented on this pull request. ------------------------------ In library/decimal.po <#2752 (comment)> : > #: ../Doc/library/decimal.rst:761 msgid "" "In the latest versions of the specification, this operation is also known as " "``reduce``." msgstr "" +"En las últimas versiones de la especificación, esta operación también era " +"conocida como ``reducir``." Hola@jcaalzago <https://github.com/jcaalzago>! Si te refieres a cómo me di cuenta de que faltaba corregir el http por https en la entrada fuzzy, eso fue simplemente porque revisando el diff me di cuenta que esa entrada ya no estaba como fuzzy, pero al mismo tiempo no habías cambiado nada, así que probablemente se te había pasado por alto algún detalle, y las URLs son lo más común que la gente no nota. Si quieres saber exactamente todos los tests que hacemos en CI puedes revisar dos archivos dentro de .github/workflows: main.yml define todos los tests automáticos que se corren en cada PR, y pr-comment.yml tiene la definición de cómo hacemos que el bot nos diga que están todas las entradas traducidas o no. Leyendo esos archivos vas a ver las distintas herramientas y scripts que usamos para las distintas tareas. De paso, te menciono que como es tu primera contribución, el repositorio no ejecuta los tests automáticamente, y los tenemos que aprobar nosotros. Pero ya después de esta primera vez todos los tests van a ejecutarse inmediatamente, por lo que vas a recibir feedback más rápidamente. Ahhh, y a todo esto, después de estos nuevos cambios que incorporaste, te va a hacer falta ejectuar powrap; luego de eso debería estar todo listo creo. — Reply to this email directly, view it on GitHub <#2752 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAVFHE4CMVJ4WJTMFTA5FADYF5TILAVCNFSM6AAAAAA7WYNBDSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBWGY3TMNBTGY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

@jcaalzago
Copy link
ContributorAuthor

Al final anoche, antes de que se lanzara el test hice lo siguiente:

  1. ejecutépowrap library/decimal.po
  2. probé a ejecutarpowrap --check --quiet library/decimal.po y ya no daba ningún error
  3. creé y subí el commitc533a42

Pero arriba ha fallado de nuevo.

Seguramente no se haga así alguno de los pasos y por eso no entra.

Ya me comentas

Gracias por todo

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.

Muchas gracias@jcaalzago por tu primer tremendo aporte, que estaba en muy buena forma! De ahora en adelante ya se ejecutarán automáticamente los tests en tus futuros PRs, por lo que obtendrás feedback más rápidamente.

Se va a producción! 🚀

@rtobarrtobar merged commit963d144 intopython:3.12Nov 25, 2023
@jcaalzago
Copy link
ContributorAuthor

Muchas gracias@jcaalzago por tu primer tremendo aporte, que estaba en muy buena forma! De ahora en adelante ya se ejecutarán automáticamente los tests en tus futuros PRs, por lo que obtendrás feedback más rápidamente.

Se va a producción! 🚀

Muchas gracias@rtobar :)

@jcaalzagojcaalzago deleted the decimal.po branchJanuary 5, 2024 11:55
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

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

Translatelibrary/decimal.po
2 participants
@jcaalzago@rtobar

[8]ページ先頭

©2009-2025 Movatter.jp