Development #64666
tox, exécuter pylint indépendamment des tests
Début:
28 avril 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Comme fait ailleurs.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 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 presque 2 ans
- Fichier 0001-wip.patch 0001-wip.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Il y a un problème qui fait que tox plante en essayant d'initialiser le deuxième environnement. Par exemple, simplement en dupliquant la conf comme dans le patch attaché (https://jenkins.entrouvert.org/job/wcs-wip/job/wip%252F64666-tox-executer-pylint-independamme/6/) :
Building wheels for collected packages: wcs Building wheel for wcs (setup.py): started Building wheel for wcs (setup.py): finished with status 'error' ERROR: Command errored out with exit status 1: command: /tmp/tox-jenkins/wcs/wip/64666-tox-executer-pylint-independamme/10/new/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-jcnwlzx8/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-jcnwlzx8/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-vvpzcvrd cwd: /tmp/pip-req-build-jcnwlzx8/ Complete output (7 lines): running bdist_wheel running build running compile_translations !!! Please install Django >= 1.4 to build translations running compile_scss running build_py error: package directory 'wcs' does not exist ---------------------------------------- ERROR: Failed building wheel for wcs
Sûrement un truc que tox essaye de réutiliser entre les environnements, peut-être qu'il faut dire à tox de ne pas, peut-être qu'il faut corriger le setup.py mais je n'y connais rien en distutils et compagnie.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution proposée à Nouveau
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-tox-move-pylint-to-separate-environment-64666.patch 0001-tox-move-pylint-to-separate-environment-64666.patch ajouté
- Statut changé de Nouveau à Solution proposée
Ça donne ça.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3a5b67b0b15dbacc94d97b8de40db4f71600c28d Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Apr 28 18:21:12 2022 +0200 tox: move pylint to separate environment (#64666)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tox: move pylint to separate environment (#64666)