Skip to content

Traducido archivo library/decimal#2752

Merged
rtobar merged 8 commits into
python:3.12from
jcaalzago:decimal.po
Nov 25, 2023
Merged

Traducido archivo library/decimal#2752
rtobar merged 8 commits into
python:3.12from
jcaalzago:decimal.po

Conversation

@jcaalzago

@jcaalzago jcaalzago commented Nov 22, 2023

Copy link
Copy Markdown
Contributor

Closes #2550

@github-actions

Copy link
Copy Markdown

All entries translated, horray! 🎉

@rtobar

rtobar commented Nov 22, 2023

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

¿Estaría ya correcto?

@jcaalzago jcaalzago changed the title Traducido archivo library/decimal Closes #2550 Traducido archivo library/decimal Nov 22, 2023

@rtobar rtobar left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment thread library/decimal.po
Comment thread library/decimal.po Outdated
Comment thread library/decimal.po Outdated
@jcaalzago

jcaalzago commented Nov 23, 2023 via email

Copy link
Copy Markdown
Contributor Author

@jcaalzago

Copy link
Copy Markdown
Contributor Author

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

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

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

@rtobar rtobar left a comment

Copy link
Copy Markdown
Collaborator

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! 🚀

@rtobar rtobar merged commit 963d144 into python:3.12 Nov 25, 2023
@jcaalzago

Copy link
Copy Markdown
Contributor Author

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 :)

@jcaalzago jcaalzago deleted the decimal.po branch January 5, 2024 11:55
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.

Translate library/decimal.po

2 participants