Skip to content

Traducido archivo binascii #568

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

Merged
merged 7 commits into from
Jul 28, 2020
Merged

Conversation

omartinex
Copy link
Contributor

Closes #556

@aroquega
Copy link
Collaborator

No te olvides de ejecutar powrap binascci.po.

Copy link
Collaborator

@aroquega aroquega left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esta bien traducido, sólo tengo 2 observaciones y 1 duda (lo de true). Es mi opinión y al final el que tiene la palabra final es @cmaureir que de seguro hará un análisis más riguroso y es su completa decisión ver si acepta o no mis comentarios.

@cmaureir
Copy link
Collaborator

Espero tu revisión entonces @hristoroque para hacer el merge, lo dejo en tus manos :)
Muchas gracias @omartinex por todo el trabajo!

@omartinex
Copy link
Contributor Author

Hola @hristoroque y @cmaureir

Es mi primera ayuda en alguna traducción, el paso a seguir es realizar los ajustes y hacer un nuevo commit sobre el mismo PR?

Gracias por la guia

@cmaureir
Copy link
Collaborator

Hola @hristoroque y @cmaureir

Es mi primera ayuda en alguna traducción, el paso a seguir es realizar los ajustes y hacer un nuevo commit sobre el mismo PR?

Gracias por la guia

Si @omartinex, a veces vamos a poner cambios acá que puedes agregar a mano con la interfaz de Github, en este caso como son generales, tienes que continuar haciendo commits en tu branch local y haciendo los push a este mismo PR.
Si tienes dudas sobre el proceso, avisanos.
Gracias a ti por la contribución!

@omartinex
Copy link
Contributor Author

Hola @cmaureir y @hristoroque

Lamento la demora, realicé algunos ajustes basado en sus comentarios

Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Con estos comentarios podemos pasar el build, a ver que te parecen.

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

Te quedan un par de palabras que agregar @omartinex en el diccionario del archivo y estaríamos 👯
https://python-docs-es.readthedocs.io/es/3.8/faq.html#como-agrego-una-palabra-al-diccionario

@omartinex
Copy link
Contributor Author

Hola @cmaureir

Creo que ya estaría listo. De nuevo perdón por la demora.

omartinex and others added 2 commits July 27, 2020 23:09
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir cmaureir merged commit 682a9ed into python:3.8 Jul 28, 2020
@cmaureir
Copy link
Collaborator

Muchas gracias @omartinex 🎉 🎉

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/binascii.po
3 participants