Skip to content

Traduccion urllib.po #236

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 6 commits into from
May 21, 2020
Merged

Traduccion urllib.po #236

merged 6 commits into from
May 21, 2020

Conversation

gomezgleonardob
Copy link
Contributor

Traduccion al español de urllib.po

@humitos
Copy link
Collaborator

humitos commented May 13, 2020

Hola @gomezgleonardob! Por algún motivo se te metió el archivo faq/installed.po en esta PR también. Deberías solo enviar el archivo de urllib.po. ¿Puedes solucionarlo?

@humitos
Copy link
Collaborator

humitos commented May 13, 2020

Por otro lado, tendrías que hacer powrap localmente para que se ajuste el ancho de columnas del archivo. Esto lo puedes hacer automáticamente instalando pre-commit en repositorio:

pip install pre-commit
pre-commit install

Luego, antes de hacer un commit, estos comandos se ejecutaran automáticamente.

@gomezgleonardob
Copy link
Contributor Author

Hola @humitos listo ya realizo los cambios e instalo precommit

@clacri
Copy link
Contributor

clacri commented May 16, 2020

hola @gomezgleonardob! gracias por el trabajo! parece que el build sigue teniendo problemas con palabras que no están en dict, concretamente:
faq/installed.po:30:Google
faq/installed.po:30:Lucasfilm
faq/installed.po:30:Ltd
faq/installed.po:85:Hewlett
faq/installed.po:85:Packard
faq/installed.po:85:Compaq
faq/installed.po:122:desinstalador

igualmente, como te recomendó @humitos una forma muy buena para hacer estos chequeos justo antes del commit es instalar pre-commit. ¿lo has podido configurar? si necesitas una ayuda con ello podemos mirarlo juntos!

@gomezgleonardob
Copy link
Contributor Author

Si mañana me puede ayudar, muchas gracias

Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

Lo marco como "Request changes" porque algo salió mal en esta PR y modifica varios archivos. Por favor, si puedes arreglarla para que solo modifique urllib.po con tu traducción sería genial. ¡Muchas gracias!

Copy link
Contributor Author

@gomezgleonardob gomezgleonardob left a comment

Choose a reason for hiding this comment

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

Fix Words/Translate

@gilgamezh
Copy link
Collaborator

gilgamezh commented May 21, 2020

@gomezgleonardob esta listo para hacer review este PR? o faltan cambios? fijate por favor que travis está fallando

Copy link
Collaborator

@gilgamezh gilgamezh left a comment

Choose a reason for hiding this comment

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

gracias!

@gilgamezh gilgamezh dismissed humitos’s stale review May 21, 2020 15:11

ya metieron los cambios :)

@gilgamezh gilgamezh merged commit f9e9963 into python:3.8 May 21, 2020
@gomezgleonardob
Copy link
Contributor Author

Listo @gilgamezh 😄

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.

4 participants