search

ctrl-alt-d / django-aula

Gestió de presencia, incidències i més en centres educatius i acadèmies.
Other
18 stars 28 forks source link

Indentar tot el codi per Black #280

Open ctrl-alt-d opened 5 days ago

ctrl-alt-d commented 5 days ago

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 ?

amorilla commented 5 days ago

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 commented 5 days ago

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

amorilla commented 4 days ago

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 commented 4 days ago

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 commented 4 days ago

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