8000 Translate 'library/i18n.po' by gfant · Pull Request #937 · python/python-docs-es · GitHub
[go: up one dir, main page]
Skip to content

Translate 'library/i18n.po' #937

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Oct 5, 2020
Merged

Translate 'library/i18n.po' #937

merged 7 commits into from
Oct 5, 2020

Conversation

gfant
Copy link
Contributor
@gfant gfant commented Oct 4, 2020

Closes #901

aroquega
aroquega reviewed Oct 4, 2020
View reviewed changes
Copy link
Collaborator
@aroquega aroquega left a comment

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.

@gfant
Copy link
Contributor Author
gfant commented Oct 4, 2020

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

aroquega
aroquega approved these changes Oct 4, 2020
View reviewed changes
Copy link
Collaborator
@aroquega aroquega left a comment

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
Copy link
Collaborator
cmaureir commented Oct 4, 2020 •
edited
Loading

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

@gfant
Copy link
Contributor Author
gfant commented Oct 4, 2020

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

@cmaureir
Copy link
Collaborator
cmaureir commented Oct 4, 2020

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
Copy link
Collaborator
cmaureir commented Oct 4, 2020

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
Copy link
Contributor Author
gfant commented Oct 4, 2020 •
edited
Loading

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
Copy link
Collaborator
cmaureir commented Oct 4, 2020

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
Copy link
Contributor Author
gfant commented Oct 4, 2020

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

image

@cmaureir
Copy link
Collaborator
cmaureir commented Oct 4, 2020

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

@gfant
Copy link
Contributor Author
gfant commented Oct 4, 2020

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

@cmaureir
Copy link
Collaborator
cmaureir commented Oct 5, 2020

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
Copy link
Collaborator
cmaureir commented Oct 5, 2020

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

@gfant
Copy link
Contributor Author
gfant commented Oct 5, 2020

¡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
Copy link
Collaborator
cmaureir commented Oct 5, 2020

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
Reviewers

@cmaureir cmaureir cmaureir approved these changes

@aroquega aroquega aroquega approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Translate 'library/i18n.po'
3 participants
@gfant @cmaureir @aroquega
0