Development #52457
passer à black et isort
0%
Description
Il faut vivre avec son temps.
Fichiers
Révisions associées
add black configuration (#52457)
misc: apply isort (#52457)
add isort configuration (#52457)
misc: remove some python2 compatibility code (#52457)
misc: remove some unused imports (#52457)
Historique
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0005-misc-remove-some-python2-compatibility-code-52457.patch 0005-misc-remove-some-python2-compatibility-code-52457.patch ajouté
- Fichier 0002-add-black-configuration-52457.patch 0002-add-black-configuration-52457.patch ajouté
- Fichier 0004-add-isort-configuration-52457.patch 0004-add-isort-configuration-52457.patch ajouté
- Fichier 0001-misc-apply-black-52457.patch 0001-misc-apply-black-52457.patch ajouté
- Fichier 0006-misc-remove-some-unused-imports-52457.patch 0006-misc-remove-some-unused-imports-52457.patch ajouté
- Fichier 0003-misc-apply-isort-52457.patch 0003-misc-apply-isort-52457.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Et voilà, j'en profite aussi pour passer 0005 et 0006 en bonus ménage (dans la lignée, manquera de virer six partout, mais ça a l'air d'être un vrai boulot).
La conf tox/jenkins vient de #50927.
Remarque, dans le passage à isort de wcs (#52224), il a été décidé de mettre --force-single-line-imports pour éviter les conflits avec black. Mais isort propose justement un --profile='black' pour éviter les conflits qu'il pourrait y avoir, je pars donc plutôt là dessus.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à En cours
Valentin Deniaud a écrit :
Et voilà, j'en profite aussi pour passer 0005 et 0006 en bonus ménage (dans la lignée, manquera de virer six partout, mais ça a l'air d'être un vrai boulot).
Ok.
La conf tox/jenkins vient de #50927.
Je préfère que black/isort soient dans une cible à part comme dans django-mellon, c'est bizarre de mélanger ça dans une cible de test. Et puis ça permet de dire aux gens dans le readme de faire tox -e code-style
ou quelque chose comme ça.
https://git.entrouvert.org/django-mellon.git/tree/tox.ini#n62
Remarque, dans le passage à isort de wcs (#52224), il a été décidé de mettre --force-single-line-imports pour éviter les conflits avec black. Mais isort propose justement un --profile='black' pour éviter les conflits qu'il pourrait y avoir, je pars donc plutôt là dessus.
Ok.
Mis à jour par Valentin Deniaud il y a environ 3 ans
Benjamin Dauvergne a écrit :
Valentin Deniaud a écrit :
La conf tox/jenkins vient de #50927.
Je préfère que black/isort soient dans une cible à part comme dans django-mellon, c'est bizarre de mélanger ça dans une cible de test. Et puis ça permet de dire aux gens dans le readme de faire
tox -e code-style
ou quelque chose comme ça.
OK je fais ça, mais pour info dans le ticket sus-cité :
Frédéric Péters a écrit :
Aussi, ça aurait peut-être pu être dans une étape jenkins explicite en plus mais avec trop de colonnes dans le tableau central d'un job jenkins le fait passer dessous, c'est moche etc.
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de En cours à Solution proposée
Voilà, branche à jour et c'est vert.
Mis à jour par Frédéric Péters il y a environ 3 ans
À propos de la combinaison black/isort, #52504, peut-être à décider avant.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
À propos de la combinaison black/isort, #52504, peut-être à décider avant.
Valentin a déjà mis --profile black, il manque le --line-length 110.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Benjamin Dauvergne a écrit :
Valentin a déjà mis --profile black, il manque le --line-length 110.
Déjà fait sur la branche, go.
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8b506d4281c85c46e4f6c30e765162bdb68e8ceb Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 11:30:15 2021 +0200 misc: remove some unused imports (#52457) commit 6c4aad3ab5c017b9226299273ab376a24423a7ee Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 11:00:52 2021 +0200 misc: remove some python2 compatibility code (#52457) commit 41e2c8eee35cdf24c1ae4ae1d2e761cff2f9bfc8 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 10:34:19 2021 +0200 add isort configuration (#52457) commit 86d946adcdc784693d733131c2aecafc6caa8cbb Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 10:38:14 2021 +0200 misc: apply isort (#52457) commit 3dc618a5716531e3b92a2bd854287038637c7534 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 10:10:04 2021 +0200 add black configuration (#52457) commit 4bb33d3d3c62516cfdb1ff5bba9216936a07d368 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 30 10:15:50 2021 +0200 misc: apply black (#52457)
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: apply black (#52457)