Skip to content

Traduccion argparse #401

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 21 commits into from
Aug 26, 2020
Merged

Traduccion argparse #401

merged 21 commits into from
Aug 26, 2020

Conversation

Qkolnek
Copy link
Contributor

@Qkolnek Qkolnek commented Jun 13, 2020

Closes #262

@cmaureir
Copy link
Collaborator

Sé que esto es un Draft, pero te recomiendo que uses tanto powrap como pospell sobre el archivo que traduces, al menos a mi me quita dolores de cabeza con el CI ;)

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Jun 16, 2020

Gracias Cristian. Imagine que Travis se quejaría, pero así voy puliendo detalles.

@clacri
Copy link
Contributor

clacri commented Jul 20, 2020

Hola @Qkolnek, quieres que se revise ya el contenido aunque esté en draft (así puedes aplicar los comentarios al resto de la traducción) o prefieres esperar? Un saludo y muchas gracias por tu colaboración y tu esfuerzo!

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Jul 23, 2020

Hola Claudia, ya tengo la traducción terminada, pero estoy realizando precisamente una revisión. Creo que para el fin de semana lo tengo listo, así que te diría que mejor esperar y ya mando todo.

@clacri
Copy link
Contributor

clacri commented Jul 23, 2020

Ningún problema :) ponlo en ready for review cuando ya esté listo o avisa por aquí mencionandome. Un saludo y muchas gracias!

@cmaureir
Copy link
Collaborator

@Qkolnek si no tienes tiempo para acabar este PR avisanos para ver si lo podemos continuar. Sería muy bueno poder tener este archivo traducido publicado.

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Aug 18, 2020

Hola Cristian, perdona que haya tardado en responder. He estado desconectado unos días, voy a tratar de terminar la revisión lo antes posible. Lo tengo terminado, como comenté, pero tengo que terminar de revisarlo.

@cmaureir
Copy link
Collaborator

Hola Cristian, perdona que haya tardado en responder. He estado desconectado unos días, voy a tratar de terminar la revisión lo antes posible. Lo tengo terminado, como comenté, pero tengo que terminar de revisarlo.

Dale! no hay problema @Qkolnek ! sólo queriamos saber como ibas :) Cualquier cosa nos dices!

@clacri
Copy link
Contributor

clacri commented Aug 23, 2020

Hola @Qkolnek en el build hay un fallo en el chequeo con pospell, lo he sacado de los detalles del build en travis:

97.85s$ pospell -p dict.txt -l es_AR -l es_ES **/.po
library/argparse.po:2320:mútuamente
The command "pospell -p dict.txt -l es_AR -l es_ES **/
.po" exited with 255.

por cierto que no sé en que sistema trabajas pero si quieres configurarte lo de pre-commit que ahora está funcionando puede venirte genial para pillar estas cosas antes de los propios commits.

En cuando al texto principal me lo reviso hoy. Un saludo!

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Voy por el párrafo 314

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Aug 23, 2020

Gracias por la revisión @clacri, me pongo con las correcciones y actualizo.

@clacri
Copy link
Contributor

clacri commented Aug 23, 2020

Tranquilo aún me queda mucho como ves lo que hago son reviews parciales hasta un párrafo determinado y te comento por donde voy para retomarlo luego. Eso sí todo lo que quieras ir mirando genial. Muchas gracias por todo!!!

@clacri clacri changed the title Traducido el 40% Traduccion argparse Aug 23, 2020
@Qkolnek
Copy link
Contributor Author

Qkolnek commented Aug 24, 2020

He hecho los cambios tal como habíamos quedado.

@clacri
Copy link
Contributor

clacri commented Aug 24, 2020

Gracias, mañana continuaré con la review. Una cosa, el build falla pero es por el problema de cpython que encontraron y comentaron ayer en telegram. Por favor, podrías hacer pull desde 3.8 para asegurarte de estar actualizado y de tener esa corrección?

@cmaureir
Copy link
Collaborator

Gracias, mañana continuaré con la review. Una cosa, el build falla pero es por el problema de cpython que encontraron y comentaron ayer en telegram. Por favor, podrías hacer pull desde 3.8 para asegurarte de estar actualizado y de tener esa corrección?

Aquí más info https://python-docs-es.readthedocs.io/es/3.8/faq.html#mantener-actualizada

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Voy por el párrafo 1298

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Habría que solucionar los errores del build respecto a los términos, ya he consultado en telegram, ahora sigo con el resto de la traducción. Encuentras la lista completa de los warnings en el build https://travis-ci.com/github/python/python-docs-es/builds/181342632

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

Te he ido dejando como sugerencias algunos de los casos

@clacri
Copy link
Contributor

clacri commented Aug 26, 2020

Ya he revisado el texto completo. El único motivo por que el que puse request changes es para que corrijas las poquitas cosas que hemos encontrado, sobre todo, el tema de los enlaces rotos que hacen fallar el build. De varios de ellos te he puesto sugerencias para q directamente hagas commit pero los otros los tendrás que sacar del build. Escríbeme si necesitas ayuda o soporte con ello. Un saludo y mil gracias por la traducción!!! Es un módulo muy interesante :)

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Aug 26, 2020

Efectivamente sabía que los xxx_ son enlaces, pero no que se perdían al añadirles *. He ido corrigiendo todos los errores que he visto en los detalles de Travis. Parece que falta algo todavía, pero trataré de arreglarlo.
Gracias @clacri por tu revisón e infinita paciencia.

@clacri
Copy link
Contributor

clacri commented Aug 26, 2020

No te preocupes ya estamos muy muy cerca :) ahora voy a ducharme y a retirarme a dormir en breve, pero si queda algo pendiente mañana lo podemos revisar!!! Gracias a ti!!! :)

@Qkolnek
Copy link
Contributor Author

Qkolnek commented Aug 26, 2020

Siiiiiiiiiii. Parece que ha pasado todo bien.

@Qkolnek Qkolnek marked this pull request as ready for review August 26, 2020 22:11
@clacri
Copy link
Contributor

clacri commented Aug 26, 2020

Me ducho y lo mergeo entonces :) (ahora estoy con la app del móvil y solo lo puedo hacer con el ordenador). Enhorabuena!

@clacri clacri merged commit 33b6bb9 into python:3.8 Aug 26, 2020
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/argparse.po
3 participants