-
Notifications
You must be signed in to change notification settings - Fork 396
Traducido archivo library/marshal #1051
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
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.
Hola @Jighdan, muchas gracias por el trabajo de traducción. Te dejo algunos comentarios a ver que te parecen.
Aparte de esto, quedará agregar términos al diccionario y, casi seguro, aplicar powrap para pasar el build.
Co-authored-by: fjsevilla <fjsevilla2.0@gmail.com>
@Jighdan pues como era de esperar... Quedan dos cositas:
Con eso debería estar contento don Travis. Si tienes cualquier problema solo comenta. |
Gracias @fjsevilla-dev |
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.
Perfecto @Jighdan, ya pasa el build sin problemas. Te dejo cuatro sugerencias que se me pasaran antes... y por mi parte se puede mergear:
Co-authored-by: fjsevilla <fjsevilla2.0@gmail.com>
@fjsevilla-dev Listo! Le aplique los cambios que sugeriste, le dare una segunda lectura al documento para ver si se me paso algo mas igual. Gracias :-) |
Gracias @Jighdan por la traducción 💪 🎉 |
Closes #874