Skip to content
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

Indentar tot el codi per Black #280

Open
ctrl-alt-d opened this issue Sep 20, 2024 · 5 comments
Open

Indentar tot el codi per Black #280

ctrl-alt-d opened this issue Sep 20, 2024 · 5 comments
Assignees

Comments

@ctrl-alt-d
Copy link
Owner

ctrl-alt-d commented Sep 20, 2024

Hola,

el codi del programa tots l'indentem com més ens agrada i no seguim una línia.

Proposo que tots fem servir Black, d'aquesta manera, ens assegurem que tots ho fem igual.

Es pot configurar Dart molt fàcilment per tal que, des de VS Code, al fer control-s, formategi el codi. Jo estic fent servir VS Code i estic content perquè les extensions de Python m'ajuden molt i a més puc debugar sense problemes. També es pot configurar per tal que ho faci al pre-commit i coses semblants i no cal ni tenir-lo integrat a l'IDE.

Llavors crec que el millor seria demanar a Dart que indenti tot el djau. Jo ho he provat i canvia 367 fiters (i deixa sense tocar 109 fitxers)

No faig la PR amb tot indentat perquè potser esteu al mig de canvis importants (per exemple en Juaky amb la versio de l'App?) i no vull crear conflictes.

Llavors, quan esteu preparats m'ho dieu i faig la PR.

Què us sembla @juaky @amorilla ?

@ctrl-alt-d ctrl-alt-d self-assigned this Sep 20, 2024
@amorilla
Copy link
Collaborator

Hola,
Jo faig servir Eclipse amb PyDev. No he trobat aquesta opció, voldria provar-ho abans.
Quan puguis, prepara una branch amb la indentació d'aquesta manera.
Salutacions.

@ctrl-alt-d ctrl-alt-d changed the title Indentar tot el codi per Dark Indentar tot el codi per Black Sep 20, 2024
@ctrl-alt-d
Copy link
Owner Author

Sorry, volia posar Black (i no Dark) he fet la PR que demana l' @amorilla a #281

@ctrl-alt-d ctrl-alt-d mentioned this issue Sep 20, 2024
@amorilla
Copy link
Collaborator

amorilla commented Sep 21, 2024

Hola,
He fet proves amb Eclipse i funciona ok.
Veig que fa diversos canvis, a més a més de la indentació, modifica espais a inici i final de parèntesis, en comentaris, canvia ' per ", ...
A mi m'ha deixat el codi igual que a la PR #281, excepte la línia 17 d'aula/apps/tutoria/views.py:

from django.template import RequestCon       # PR withdart. Error.
from django.template import RequestContext   # master, black amb Eclipse. Ok.

Em sembla bé fer servir aquest format, però s'ha de verificar que ha passat en aquesta línia.

Salutacions.

Posteriorment, he vist que el RequestContext no es fa servir per a res, però igualment no s'hauria d'haver modificat. En altres fitxers ha anat bé.

@ctrl-alt-d
Copy link
Owner Author

Crec que aquest error l'he introduït jo a ma. Sorry.

A veure que diu en @juaky . No sé si ell té una branca amb el tema de l'app i si ara fem el canvi a Black seria potser un maldecap per ell.

@juaky
Copy link
Collaborator

juaky commented Sep 21, 2024

Hola.
Jo utilitzo Pycharm. Acabo d'instal·lar el Black.
Per mi ok.
Això sí, tal i com diu en Dani, hi ha una branca que està pendent de fer PR. La tinc en Draft perquè encara s'estan fent proves.
En el moment que les proves siguin positives i pugui fer el merge, endavant amb Black

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

No branches or pull requests

3 participants