Développement #52457
passer à black et isort
0%
Description
Il faut vivre avec son temps.
Files
Associated revisions
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)
History
Updated by Valentin Deniaud over 3 years ago
- File 0005-misc-remove-some-python2-compatibility-code-52457.patch 0005-misc-remove-some-python2-compatibility-code-52457.patch added
- File 0002-add-black-configuration-52457.patch 0002-add-black-configuration-52457.patch added
- File 0004-add-isort-configuration-52457.patch 0004-add-isort-configuration-52457.patch added
- File 0001-misc-apply-black-52457.patch 0001-misc-apply-black-52457.patch added
- File 0006-misc-remove-some-unused-imports-52457.patch 0006-misc-remove-some-unused-imports-52457.patch added
- File 0003-misc-apply-isort-52457.patch 0003-misc-apply-isort-52457.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Benjamin Dauvergne over 3 years ago
- Status changed from Solution proposée to 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.
Updated by Valentin Deniaud over 3 years ago
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-styleou 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.
Updated by Valentin Deniaud over 3 years ago
- Status changed from En cours to Solution proposée
Voilà, branche à jour et c'est vert.
Updated by Frédéric Péters over 3 years ago
À propos de la combinaison black/isort, #52504, peut-être à décider avant.
Updated by Benjamin Dauvergne over 3 years ago
- Status changed from Solution proposée to 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.
Updated by Benjamin Dauvergne over 3 years ago
Benjamin Dauvergne a écrit :
Valentin a déjà mis --profile black, il manque le --line-length 110.
Déjà fait sur la branche, go.
Updated by Valentin Deniaud over 3 years ago
- Status changed from Solution validée to 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)Updated by Frédéric Péters over 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: apply black (#52457)