Development #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.
Fichiers
Demandes liées
Révisions associées
misc: add djhtml files/notes (#69404)
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-misc-apply-djhtml-69404.patch 0001-misc-apply-djhtml-69404.patch ajouté
- Fichier 0002-misc-add-djhtml-files-notes-69404.patch 0002-misc-add-djhtml-files-notes-69404.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Marche bien, rapide comme l'éclair.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Moi ça me va mais je veux bien un deuxième avis.
Mis à jour par Valentin Deniaud il y a plus d'un an
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.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à 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)Mis à jour par Valentin Deniaud il y a plus d'un an
- Lié à Development #69422: pre-commit-hooks, utiliser djhtml partout ajouté
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: apply djhtml (#69404)