Development #52797
Intégrer isort
0%
Description
intégrer isort, avec hook pre-commit
Fichiers
Révisions associées
misc: apply isort (#52797)
misc: add isort reformat reference to git blame ignore list (#52797)
Historique
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0002-misc-apply-isort-52797.patch 0002-misc-apply-isort-52797.patch ajouté
- Fichier 0001-misc-add-isort-52797.patch 0001-misc-add-isort-52797.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a environ 3 ans
Et donc il manquerait un troisième commit qui vient ajouter le hash du second dans .git-blame-ignore-revs
?
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0003-misc-add-isort-reformat-reference-to-git-blame-ignor.patch 0003-misc-add-isort-reformat-reference-to-git-blame-ignor.patch ajouté
exact, merci :)
Mis à jour par Paul Marillonnet il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Nickel.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-misc-add-isort-52797.patch 0001-misc-add-isort-52797.patch ajouté
- Statut changé de Solution validée à Solution proposée
(avec update de tox.ini)
Mis à jour par Paul Marillonnet il y a environ 3 ans
Et donc il manque aussi un s/black-pylint/codestyle/
sur le nom des environnement de tests définis au début du fichier tox.ini
, non ?
Mis à jour par Lauréline Guérin il y a environ 3 ans
ha ben oui, en effet :)
j'ai vérifié le fichier jenkins mais pas le début du fichier tox ..
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-misc-add-isort-52797.patch 0001-misc-add-isort-52797.patch ajouté
Mis à jour par Paul Marillonnet il y a environ 3 ans
Jenkins il trouve pas pre-commit
:
ERROR: InvocationError for command could not find executable 'pre-commit'
On pourrait ajouter une dépendance spécifique à l’environnement de test, comme par exemple ce qui a été fait dans authentic2.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-misc-add-isort-52797.patch 0001-misc-add-isort-52797.patch ajouté
(comme wcs et chrono)
Mis à jour par Paul Marillonnet il y a environ 3 ans
Lauréline Guerin a écrit :
(comme wcs et chrono)
Ok, juste j’aurais bien aimé voir Jenkins vert pour voir qu’il est d’accord avec nous, tu peux d’abord pousser dans cette branche avec le commit à jour stp ?
(Et, avant d’envoyer tout dans main, ne pas oublier de mettre à jour 0003 avec le nouveau hash de 0002.)
Mis à jour par Paul Marillonnet il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Cool, ça tourne comme il faut. Je te laisse modifier 0003 pour le hash à jour et c’est bon pour moi.
Mis à jour par Paul Marillonnet il y a environ 3 ans
Paul Marillonnet a écrit :
Je te laisse modifier 0003 pour le hash à jour et c’est bon pour moi.
Et j’avais pas vu que c’était déjà fait, mes excuses.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4021e5d3b0b49e0e011ed11751b033ba34d4e748 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Apr 8 09:28:18 2021 +0200 misc: add isort reformat reference to git blame ignore list (#52797) commit 29bc8e66a978b1bbadbc05186599a70ce1b8ef98 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Apr 6 21:31:24 2021 +0200 misc: apply isort (#52797) commit 6c6d7bac407d7ff697dc54c63723c1b051d24a02 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Tue Apr 6 21:28:51 2021 +0200 misc: add isort (#52797)
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: add isort (#52797)