Développement #69404
pre-commit-hooks, utiliser djhtml
0%
Description
Là aussi évoqué et jamais mis en place.
De manière générale le formattage de nos templates est en piteux état, malheuseument il n'existe aucun outil style black pour les reformatter (prettier que tout le monde utilise ne supportera jamais Django (https://github.com/prettier/prettier/issues/5581#issuecomment-647403815), et si d'autres projets existent rien n'a l'air très mature).
Le seul outil qui a l'air stable c'est djhtml, ça ne regarde que l'indentation (pas de joli split des lignes de 500 caractères qu'on peut avoir ici ou là) mais on y gagnera déjà pas mal.
Files
Related issues
Associated revisions
misc: add djhtml files/notes (#69404)
History
Updated by Valentin Deniaud about 2 years ago
- File 0001-misc-apply-djhtml-69404.patch 0001-misc-apply-djhtml-69404.patch added
- File 0002-misc-add-djhtml-files-notes-69404.patch 0002-misc-add-djhtml-files-notes-69404.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Marche bien, rapide comme l'éclair.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to Solution validée
Moi ça me va mais je veux bien un deuxième avis.
Updated by Valentin Deniaud about 2 years ago
Frédéric Péters a écrit :
Ça tourne correctement sur tous les modules ?
Dans ma tête c'était activation sur chrono pour tester en condition réelles, puis ticket chapeau dans le projet Publik pour passage sur tous les modules, puis passage sur tous les modules.
Ça ne me pose pas de soucis de faire autrement, je vais faire le ticket équivalent côté wcs pour confirmer.
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 6a8ebfcfd23b8839415e980cb30e418ffa74a88b
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Sep 21 17:37:13 2022 +0200
misc: add djhtml files/notes (#69404)
commit 7c0eb04c453f50cdffe0095148aa52580c20143e
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Sep 21 17:28:29 2022 +0200
misc: apply djhtml (#69404)Updated by Valentin Deniaud about 2 years ago
- Related to Développement #69422: pre-commit-hooks, utiliser djhtml partout added
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: apply djhtml (#69404)