Development #63008
tox, exécuter pylint indépendamment des tests
0%
Description
Pour que jenkins les rapporte même quand un test plante.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a environ 2 ans
- Lié à Development #61397: Builds jenkins, lancer pylint même si les tests ne passent pas ajouté
Mis à jour par Valentin Deniaud il y a environ 2 ans
- Fichier 0001-tox-move-pylint-to-separate-environment-63008.patch 0001-tox-move-pylint-to-separate-environment-63008.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Build en échec qui montre le bon fonctionnement https://jenkins.entrouvert.org/job/chrono-wip/job/wip%252F63008-tox-executer-pylint-independamme/3/
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
Ya besoin de toutes ces dépendances pour exécuter pylint ?
Mis à jour par Lauréline Guérin il y a environ 2 ans
pour éviter certaines erreurs de type import-error
, oui
Mis à jour par Frédéric Péters il y a environ 2 ans
Et il n'y a pas de truc pour partager avec la liste définie/maintenue dans testenv, duplication obligée ?
Aussi, la partie usedevelop = coverage: True / nocoverage: False, a priori pas besoin de la reprendre, et pareil pour certaines lignes (toutes?) de setenv ?
Mis à jour par Valentin Deniaud il y a environ 2 ans
- Fichier 0001-tox-move-pylint-to-separate-environment-63008.patch 0001-tox-move-pylint-to-separate-environment-63008.patch ajouté
Lauréline Guerin a écrit :
pour éviter certaines erreurs de type
import-error
, oui
J'ai refait tourner un build pour le démontrer, https://jenkins.entrouvert.org/job/chrono-wip/job/wip%252F63008-tox-executer-pylint-independamme/5/pylint/new/fileName.-1357168269/source.4298e3c7-75a0-4c0b-9d4a-5355e010d9b2/#5.
Frédéric Péters a écrit :
Et il n'y a pas de truc pour partager avec la liste définie/maintenue dans testenv, duplication obligée ?
Il n'y en avait pas en août 2021 https://github.com/tox-dev/tox/discussions/2193.
Aussi, la partie usedevelop = coverage: True / nocoverage: False, a priori pas besoin de la reprendre, et pareil pour certaines lignes (toutes?) de setenv ?
Viré et effectivement c'est toujours vert, merci.
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d4f05d10a4d5d7e3f0c5ce2f523bcc2730d8ec57 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Mar 21 16:00:12 2022 +0100 tox: move pylint to separate environment (#63008)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tox: move pylint to separate environment (#63008)