Development #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.
Fichiers
Révisions associées
misc: remove excessive spaces from strings (#56007)
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0002-misc-apply-black-56007.patch 0002-misc-apply-black-56007.patch ajouté
- Fichier 0001-pre-commit-update-black-and-add-string-processing-op.patch 0001-pre-commit-update-black-and-add-string-processing-op.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 2 ans
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 ?
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-concatenate-strings-when-possible-56007.patch 0001-misc-concatenate-strings-when-possible-56007.patch ajouté
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.
Mis à jour par Frédéric Péters il y a plus de 2 ans
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.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-concatenate-strings-when-possible-56007.patch 0001-misc-concatenate-strings-when-possible-56007.patch ajouté
- Fichier 0002-misc-remove-excessive-spaces-from-strings-56007.patch 0002-misc-remove-excessive-spaces-from-strings-56007.patch ajouté
Rebasé, et 0002 pour passer un coup de sed qui vire des espaces en trop rendus apparents dans le diff de 0001.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: concatenate strings when possible (#56007)