Développement #56007
Utiliser black avec --experimental-string-processing
0%
Description
En reformattant le code black a laissé plein de chaînes en mauvais état, genre
raise serializers.ValidationError(
_('you are not authorized ' 'to create users in ' 'this ou')
)
Or il y a une option pour que les chaînes soient concaténées (+ qu'elles respectent le paramètre line-length), un peu de lecture de github dit que ça a été introduit par défaut il y a un an et demi, puis mis derrière un flag parce que sujet à bug, et que maintenant c'est plutôt tout bon pour que ce soit remis par défaut, https://github.com/psf/black/issues/2188.
En tout cas ça semble tourner sans problème sur le code d'authentic.
Files
Associated revisions
misc: remove excessive spaces from strings (#56007)
History
Updated by Valentin Deniaud over 3 years ago
- File 0002-misc-apply-black-56007.patch 0002-misc-apply-black-56007.patch added
- File 0001-pre-commit-update-black-and-add-string-processing-op.patch 0001-pre-commit-update-black-and-add-string-processing-op.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters over 3 years ago
Il y a moyen de l'appliquer un coup pour corriger les chaines, mais de ne pas l'intégrer dans le pre-commit, histoire de conserver le même fichier pre-commit dans tous les projets ?
Updated by Valentin Deniaud over 3 years ago
- File 0001-misc-concatenate-strings-when-possible-56007.patch 0001-misc-concatenate-strings-when-possible-56007.patch added
Frédéric Péters a écrit :
Il y a moyen de l'appliquer un coup pour corriger les chaines, mais de ne pas l'intégrer dans le pre-commit, histoire de conserver le même fichier pre-commit dans tous les projets ?
Sure thing, je me disais juste que ça pouvait devenir une option à ajouter dans les autres projets.
Updated by Frédéric Péters over 3 years ago
Sure thing, je me disais juste que ça pouvait devenir une option à ajouter dans les autres projets.
Je suis plutôt pour attendre que ça intègre le comportement par défaut.
~~
En fait j'imaginais que ça ferait juste les corrections type
- help_text=_('Designates whether the user can log into this admin ' 'site.'),
+ help_text=_('Designates whether the user can log into this admin site.'),
mais ça coupe aussi les trop longues chaines, façon
- 'CREATE INDEX journal_event_reference_ct_ids_idx ON journal_event USING GIN("reference_ct_ids");',
+ 'CREATE INDEX journal_event_reference_ct_ids_idx ON journal_event USING'
+ ' GIN("reference_ct_ids");',
j'avais raté le "(+ qu'elles respectent le paramètre line-length)" de la description; ça passe quand même ainsi, la relecture du diff permet de relever au moins un bug mais pas lié à black.
Updated by Valentin Deniaud over 3 years ago
- File 0001-misc-concatenate-strings-when-possible-56007.patch 0001-misc-concatenate-strings-when-possible-56007.patch added
- File 0002-misc-remove-excessive-spaces-from-strings-56007.patch 0002-misc-remove-excessive-spaces-from-strings-56007.patch added
Rebasé, et 0002 pour passer un coup de sed qui vire des espaces en trop rendus apparents dans le diff de 0001.
Updated by Benjamin Dauvergne over 3 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud over 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 56972ed0da94e75b61bbfc042894d52d6979a39b
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Aug 10 11:02:04 2021 +0200
misc: remove excessive spaces from strings (#56007)
commit f24096f8b8ccf119e914494e254f07b3e169f626
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Aug 10 10:47:28 2021 +0200
misc: concatenate strings when possible (#56007)Updated by Frédéric Péters over 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: concatenate strings when possible (#56007)