Skip to content

Translate 'library/i18n.po'#937

Merged
cmaureir merged 7 commits into
python:3.8from
gfant:traduccion-i18n
Oct 5, 2020
Merged

Translate 'library/i18n.po'#937
cmaureir merged 7 commits into
python:3.8from
gfant:traduccion-i18n

Conversation

@gfant

@gfant gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor

Closes #901

@aroquega aroquega left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Muy bien la traducción aunque fuera corta, sólo puse comentarios que creo que mejorarían la traducción un poco.

Comment thread library/i18n.po Outdated
Comment thread library/i18n.po Outdated
Comment thread library/i18n.po Outdated
@gfant

gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor Author

@hristoroque ya ha sido corregido en un commit cada uno de los detalles que mencionas. Cuando tengas tiempo, por favor revísalos :-)

@aroquega aroquega left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Está bien la traducción 👍🥳

@cmaureir

cmaureir commented Oct 4, 2020

Copy link
Copy Markdown
Collaborator

falta pasar un powrap y estamos @iam-agf ;)
powrap library/i18n.po

@gfant

gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor Author

@cmaureir qué es un powrap y cómo lo puedo realizar?

@cmaureir

cmaureir commented Oct 4, 2020

Copy link
Copy Markdown
Collaborator

https://python-docs-es.readthedocs.io/es/3.8/CONTRIBUTING.html powrap y pospell son dos herramientas que recomendamos.
powrap, ayuda a utilizar una cantidad definida de columnas por archivos, es decir, acorta las líneas muy largas.
pospell lo que hace es mirar la ortografía de los archivos.

@cmaureir

cmaureir commented Oct 4, 2020

Copy link
Copy Markdown
Collaborator

La verificación que tenemos, puedes ver que eso:

 powrap --check --quiet **/*.po
Would rewrap:
- library/i18n.po
The command "powrap --check --quiet **/*.po" exited with 1.

Entonces solo lo puedes resolver instalando powrap en tu entorno virtual y luego solo ejecutarlo en tu archivo powrap library/i18n.po y subir los cambios del archivo.

@gfant

gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor Author

Instalé powrap ejecuté el comando que mencionaste, powrap library/i18n.po e hice el push con el nombre powrap fix, pero sigue sin completarse ¿qué otra cuestión puede faltar?

image

@cmaureir

cmaureir commented Oct 4, 2020

Copy link
Copy Markdown
Collaborator

Hiciste todos estos pasos @iam-agf ?

powrap library/i18n.po
git add library/i18n.po
git commit -m "powrap i18n"
git push origin traduccion-i18n

@gfant

gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor Author

@cmaureir sí, justamente por si acaso acabo de volver a realizarlo:

image

@cmaureir

cmaureir commented Oct 4, 2020

Copy link
Copy Markdown
Collaborator

Quizás la culpa es de Windows y Git-bash :o
2020-10-04-224547_1296x521_scrot

@gfant

gfant commented Oct 4, 2020

Copy link
Copy Markdown
Contributor Author

Perdón, no entendí bien la diferencia. ¿Qué hizo distinto que resultó en algo distinto entonces?

@cmaureir

cmaureir commented Oct 5, 2020

Copy link
Copy Markdown
Collaborator

Perdón, no entendí bien la diferencia. ¿Qué hizo distinto que resultó en algo distinto entonces?

El funcionamiento de Linux y Windows parece ser distinto, y como el CI que tenemos (Travis) es un Linux, al parecer los usuarios de Windows no tienen la misma cantidad de columnas por defecto en powrap.

@cmaureir cmaureir merged commit 71c1e75 into python:3.8 Oct 5, 2020
@cmaureir

cmaureir commented Oct 5, 2020

Copy link
Copy Markdown
Collaborator

Pero ya lo tienes @iam-agf ;) muchas gracias 🎉

@gfant

gfant commented Oct 5, 2020

Copy link
Copy Markdown
Contributor Author

¡Gracias! Aprendí muchísimo cuando empecé a programar con Python, entonces al ver que podía aportar algo acá, aunque sea poquito, es un sueño :-).

@cmaureir

cmaureir commented Oct 5, 2020

Copy link
Copy Markdown
Collaborator

Pues que sepas que esta es "una contribución oficial a Python", ya que estamos en la organización de github ;) así que imagina el impacto de tu aporte!

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/i18n.po'

3 participants