-
Notifications
You must be signed in to change notification settings - Fork 396
Traducción distutils/apiref.po #756
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
…om-samvaldez/traduccion-apiref traduccion-apiref
@samvaldez tendrías que haber aplicado los cambios en el mismo Pull-Request :( ahora tenemos también #729 y no podemos revisar este, pues depende de los otros. Veo que estás subiendo archivos a github por un modo de subir archivos, si tienes problemas con usar git, deberías unirte a nuestro grupo de telegram para poder ir aclarando todas tus dudas. Entonces, este parche incluye lo que habías hecho en #729 ? si es así, podemos cerrar ese y revisar este. |
Lo otro, tienes el archivo aplicado dos veces y también modificaste la lista de todos los TRANSLATORS. Deberias quitar el archivo 'apiref.po' y dejar 'distutils/apiref.po'. |
Lo hago desde el command line de git, pero al parecer no lo estoy haciendo correctamente. Pero si, creo que necesitare ayuda. Si, este parche incluye lo que había hecho en #729. |
No pasa nada @samvaldez :) Si te hacen cualquier comentario te recomiendo usar la interfaz de github para ir agregando los cambios y comentarios, son solo clicks, asi que debería ser más sencillo. Si quieres usar la terminal, recuerda hacer un 'git pull' de tu rama local llamada 'cambios', así vas a poder actualizar locamente lo que he hecho para que quede mejor el PR. |
De acuerdo, muchas gracias 👍🏻 :) |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
A ver si alguien nos da una mano revisando el archivo ahora :) si no, quizás en unos días pueda leerlo completo. Gracias por tu paciencia, queda poco para poder hacer el merge \o/ |
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.
Revisado hasta la línea 708, mañana lo continúo, si nadie más lo hace.
@samvaldez dejé unos comentarios, para cuando tengas un tiempito :)
distutils/apiref.po
Outdated
"The ``setuptools`` project adds new capabilities to the ``setup`` function " | ||
"and other APIs, makes the API consistent across different Python versions, " | ||
"and is hence recommended over using ``distutils`` directly." | ||
"The ``setuptools`` project adds new capabilities to the ``setup`` function and " |
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 parte en inglés no se deberían tocar no?
@samvaldez échale un vistazo a las sugerencias que te han dejado! 👯 |
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Co-authored-by: Emmanuel Arias <eamanu@yaerobi.com>
Quedan un par de problemas que resolver que pospell notó:
|
Gracias @samvaldez 🎉 |
Correcciones aplicadas a la traducción del archivo apiref según sugerencias.
Closes #478