Skip to content

Traduciendo classes#50

Merged
gilgamezh merged 4 commits into
python:3.7from
marcorichetta:tutorial_clases
May 5, 2020
Merged

Traduciendo classes#50
gilgamezh merged 4 commits into
python:3.7from
marcorichetta:tutorial_clases

Conversation

@marcorichetta

@marcorichetta marcorichetta commented May 4, 2020

Copy link
Copy Markdown
Contributor

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

@humitos

humitos commented May 4, 2020

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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

humitos commented May 4, 2020

Copy link
Copy Markdown
Collaborator

@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
Copy Markdown
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

humitos commented May 4, 2020

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Contributor Author

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

@gilgamezh gilgamezh 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.

@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

@gilgamezh gilgamezh 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.

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