Skip to content

Traduciendo classes #50

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

Traduciendo classes #50

merged 4 commits into from
May 5, 2020

Conversation

marcorichetta
Copy link
Contributor

@marcorichetta marcorichetta commented May 4, 2020

Veo que hay diferencia con la cantidad de caracteres por línea.
Por defecto tengo 79 en Poedit. Hay problema con esto?

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

Veo que hay diferencia con la cantidad de caracteres por línea.
Por defecto tengo 79 en Poedit. Hay problema con esto?

@marcorichetta puedes usar el comando powrap para arreglar eso. Debería dejar las líneas con la misma cantidad de columnas que estamos usando en el repositorio.

https://pypi.org/project/powrap/

¿Querés probarlo y hacer un commit + push de esos cambios?

@marcorichetta
Copy link
Contributor Author

¿Querés probarlo y hacer un commit + push de esos cambios?

Se me pasó lo de powrap. 🤦

Lo ejecuté con powrap tutorial/classes.po sin argumentos.

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

@marcorichetta genial.

Por cierto, hay un commando de pre-commit (https://pre-commit.com/) ya en el repositorio (si quieres usar eso), pero todavía no hemos documentado como se usa.

Básicamente, ejecuta powrap automaticamente antes de hacer el commit y te arregla los archivos solo.

Los pasos para usarlo serían, más o menos:

pip install pre-commit
pre-commit install

Luego con cada git commit se va a ejecutar automáticamente.

@marcorichetta
Copy link
Contributor Author

@humitos Ahí lo instalé y configuré asi no pasa de nuevo jaja

Esto no está en el CONTRIBUTING por algún motivo?

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

Esto no está en el CONTRIBUTING por algún motivo?

Sí. Básicamente, no queremos complicarle la vida al que quiere colaborar con tantas cosas técnicas antes de empezar. Lo ideal sería hacer #51

@marcorichetta marcorichetta changed the title Traduciendo classes (WIP) Traduciendo classes May 4, 2020
@marcorichetta
Copy link
Contributor Author

Recién veo que Sergio también estaba traduciendo classes.

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.

@marcorichetta está ok. fijate de resolver los conflictos y lo mergeamos

@marcorichetta marcorichetta requested review from gilgamezh and removed request for humitos and raulcd May 4, 2020 21:58
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 merged commit 2366ab9 into python:3.7 May 5, 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.

3 participants