Traducido archivo library/decimal #2752
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? |
jcaalzago
changed the title
There was a problem hiding this comment.
@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.
|
Muchas gracias,
probablemente falte hacer el powrap, me dio un error de rewarp al
subirlo... y he subido otra revisión antes de ver este comentario.... me
imagino que no pasará el test...
Si no lo pasa de nuevo, lo hago y lo subo.
Muchas gracias por todas las indicaciones :)
Saludos
El 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.
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 :) |
Closes #2550