Skip to content
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

Traducción distutils/apiref.po #756

Merged
merged 26 commits into from Dec 20, 2020
Merged

Traducción distutils/apiref.po #756

merged 26 commits into from Dec 20, 2020

Conversation

@samvaldez
Copy link
Contributor

@samvaldez samvaldez commented Sep 15, 2020

Correcciones aplicadas a la traducción del archivo apiref según sugerencias.
Closes #478

samvaldez added 3 commits Aug 18, 2020
…om-samvaldez/traduccion-apiref

traduccion-apiref
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Sep 17, 2020

@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.

@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Sep 17, 2020

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'.
Pero ten cuidado, no sé cual de los dos has cambiado.

@samvaldez
Copy link
Contributor Author

@samvaldez samvaldez commented Sep 18, 2020

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.

@cmaureir cmaureir changed the title Corrección del archivo traducido distutils/apiref.po Traducción distutils/apiref.po Sep 20, 2020
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Sep 20, 2020

No pasa nada @samvaldez :)
He cerrado el otro PR y acá he hecho un revert de uno de tus commits, que traia el archivo que no toca 'apiref.po' pues debía estar dentro del directorio 'distutils/'.
Ahora lo que hice fue dejar solo el archivo distutils/apiref.po y agregar tu nombre a la lista de TRANSLATORS pues antes había un problema extraño y aparecía que habias borrado todos los nombres y los habías agregado de nuevo.

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.

distutils/apiref.po Outdated Show resolved Hide resolved
cmaureir added 3 commits Sep 20, 2020
This reverts commit 27882d1.
@samvaldez
Copy link
Contributor Author

@samvaldez samvaldez commented Sep 21, 2020

De acuerdo, muchas gracias 👍🏻 :)

Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Sep 21, 2020

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/

Copy link
Contributor

@eamanu eamanu left a comment

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

"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 "

This comment has been minimized.

@eamanu

eamanu Sep 27, 2020
Contributor

Esta parte en inglés no se deberían tocar no?

distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
distutils/apiref.po Outdated Show resolved Hide resolved
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Oct 4, 2020

@samvaldez échale un vistazo a las sugerencias que te han dejado! 👯

samvaldez and others added 10 commits Oct 10, 2020
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
Co-authored-by: Emmanuel Arias <[email protected]>
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Oct 10, 2020

Quedan un par de problemas que resolver que pospell notó:

$ pospell -p dict.txt -l es_AR -l es_ES **/*.po

<rst-doc>:1: (WARNING/2) Inline literal start-string without end-string. while parsing: *option_table* es una lista de 3 tuplas: ``(*long_option*, *short_option*, *help_string*)`
<rst-doc>:1: (WARNING/2) Inline interpreted text or phrase reference start-string without end-string. while parsing: Todas las opciones son booleanas y afectan los valores retornados por :meth:`readline
distutils/apiref.po:253:estan
distutils/apiref.po:400:despúes
distutils/apiref.po:413:despúes
distutils/apiref.po:425:despúes
distutils/apiref.po:438:despúes
distutils/apiref.po:499:commando
distutils/apiref.po:529:definirla
distutils/apiref.po:655:extension
distutils/apiref.po:679:libpython
distutils/apiref.po:744:vinculador
distutils/apiref.po:913:compiladorde
distutils/apiref.po:939:vinculador
distutils/apiref.po:1003:indefinidad
distutils/apiref.po:1450:Preprocesa
distutils/apiref.po:1620:fué
distutils/apiref.po:1669:cygwin
distutils/apiref.po:1795:commandos
distutils/apiref.po:1958:configuralo
distutils/apiref.po:2875:unela
distutils/apiref.po:2947:despues
distutils/apiref.po:3047:edítelo
distutils/apiref.po:3173:Installer
distutils/apiref.po:3197:Redhat
cmaureir added 8 commits Nov 21, 2020
@cmaureir cmaureir merged commit d28fa7f into python:3.8 Dec 20, 2020
2 checks passed
2 checks passed
Test
Details
docs/readthedocs.org:python-docs-es Read the Docs build succeeded!
Details
@cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Dec 20, 2020

Gracias @samvaldez 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.