-
Notifications
You must be signed in to change notification settings - Fork 396
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
Conversation
No te olvides de ejecutar |
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.
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.
Espero tu revisión entonces @hristoroque para hacer el merge, lo dejo en tus manos :) |
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. |
Hola @cmaureir y @hristoroque Lamento la demora, realicé algunos ajustes basado en sus comentarios |
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.
Con estos comentarios podemos pasar el build, a ver que te parecen.
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Te quedan un par de palabras que agregar @omartinex en el diccionario del archivo y estaríamos 👯 |
Hola @cmaureir Creo que ya estaría listo. De nuevo perdón por la demora. |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Muchas gracias @omartinex 🎉 🎉 |
Closes #556