- Notifications
You must be signed in to change notification settings - Fork396
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
All entries translated, horray! 🎉 |
@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! |
¿Estaría ya correcto? |
There was a problem hiding this 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.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
jcaalzago commentedNov 23, 2023 via email• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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: ***@***.***> |
Al final anoche, antes de que se lanzara el test hice lo siguiente:
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 |
There was a problem hiding this 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! 🚀
Muchas gracias@rtobar :) |
Uh oh!
There was an error while loading.Please reload this page.
Closes#2550