Skip to content

Translation unix po #267

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

Translation unix po #267

merged 7 commits into from
May 21, 2020

Conversation

pandrearro
Copy link
Contributor

Traducción de archivo library/unix.po

Closes #244

@clacri
Copy link
Contributor

clacri commented May 16, 2020

Hola @pandrearro gracias por la traducción! El build está fallando por el powrap check, que es el que mira el tamaño de las líneas. Esto se puede ajustar automáticamente previo al comit usando el comando:
powrap library/unix.po
Si no tienes instalado powrap puede hacerse via pip (pip install powrap). También, tenemos unos hooks de pre-commit muy buenos que permiten hacer automaticamente estos chequeos antes del commit. Para instalar pre-commit, tambien puede hacerse con pip. Para que se configure en tu repositorio, tienes que meterte en el y ejecutar:
pre-commit install

Si necesitas ayuda para configurarlo, aquí estamos :)

@pandrearro
Copy link
Contributor Author

Hola @clacri , estuve intentando ejecutar ambas opciones pero para Powrap me sale este error "TypeError: argument of type 'WindowsPath' is not iterable", ya que estoy trabajando en Windows y con los Hooks me sale esto "An error has occurred: FatalError: git failed. Is it installed, and are you in a Git repository directory?" estando dentro de la carpeta donde hice el clone del repo.

Que tendria que tener en cuenta al momento de hacer el commit para ver si puedo corregirlo directamente sobre el archivo unix.po? Gracias.

@clacri
Copy link
Contributor

clacri commented May 17, 2020

Hola @pandrearro :) No trabajo con windows habitualmente y menos en términos de programación en Python, pero igual podemos poner en común con otra gente del grupo (se que hay varias personas que usan windows) para ver si han tenido problemas parecidos. De todas formas, por tu mensaje entiendo que conseguiste instalar ambas herramientas (powrap y pre-commit) pero lo que te falla son sus ejecuciones.

Igualmente, si quieres escríbeme por telegram (soy Claudia allí) y intentamos mirar juntas como solucionar esos dos errores. El único error que te da el build, si te fijas, es el de powrap, no se queja sobre pospell, y esto me temo que realmente la solución más fácil es pasarle el powrap. Siempre podría pasartelo otra persona pero lo mejor es que consigamos que te funcione la herramienta :)

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.

Muchas gracias por tu traducción! a ver si conseguimos solucionar lo de powrap y podemos mergear :)

@clacri clacri merged commit fabf462 into python:3.8 May 21, 2020
@humitos
Copy link
Collaborator

humitos commented May 21, 2020

Yaaaay! 🎉 Muchas gracias por el trabajo que hicieron en esta PR 😄

@pandrearro pandrearro deleted the translation-unix-po branch May 21, 2020 14:09
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/unix.po
4 participants