Development #52224
Intégrer isort
0%
Description
Un peu comme black, ne plus avoir à se soucier de la présentation des imports; isort le fait à notre place
Fichiers
Historique
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0002-misc-apply-isort-52224.patch 0002-misc-apply-isort-52224.patch ajouté
- Fichier 0001-misc-use-isort-52224.patch 0001-misc-use-isort-52224.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 environ 3 ans
- Statut changé de Solution proposée à Solution validée
isort is used to format the imports, using those parameter:
parameterS.
Concernant l'exécution jenkins, peut-êre changer le nom et intégrer les deux dans le même pre-commit run ?
genre
-envlist = py3-django111-black-isort-pylint-coverage, py3-django22 +envlist = py3-django111-codestyle-pylint-coverage, py3-django22 ... - black: pre-commit run black --all-files --show-diff-on-failure - isort: pre-commit run isort --all-files --show-diff-on-failure + codestyle: pre-commit run --all-files --show-diff-on-failure
Comme tu préfères.
Mis à jour par Frédéric Péters il y a environ 3 ans
(cela étant je ne suis quand même pas bien fan de cette interdiction de plusieurs imports par ligne, je me passerais volontiers de --force-single-line-imports, mais je me dis que ça fait peut-être des conflits avec ce que black produit etc.).
Mis à jour par Lauréline Guérin il y a environ 3 ans
oui j'ai oublié de préciser:
black produit des imports comme ça:
import ( truc, muche, chouette, )
alors que isort, avec l'option qui va bien pour mettre à la ligne à peu près de la même façon, vire la dernière virgule. Ou inversement, je sais plus :)
Bref, pour éviter que l'un invalide l'autre, j'ai préféré un import par ligne
Mis à jour par Frédéric Péters il y a environ 3 ans
Ok j'imaginais bien une affaire de ce genre.
Je suis pour merger ça au plus vite, pour éviter du travail de rebase plus tard.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 48470c50c0e9b56e989cfbcdcec433de0cc8479a Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 14:41:24 2021 +0100 misc: apply isort (#52224) commit 3095510966f12750e6402138b6cab54337cca501 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 11:34:18 2021 +0100 misc: use isort (#52224)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: use isort (#52224)