Développement #64666
tox, exécuter pylint indépendamment des tests
Start date:
28 April 2022
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
Comme fait ailleurs.
Files
Related issues
Associated revisions
History
Updated by Valentin Deniaud over 2 years ago
- Related to Développement #61397: Builds jenkins, lancer pylint même si les tests ne passent pas added
Updated by Valentin Deniaud over 2 years ago
- File 0001-wip.patch 0001-wip.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Valentin Deniaud over 2 years ago
- File 0001-tox-move-pylint-to-separate-environment-64666.patch 0001-tox-move-pylint-to-separate-environment-64666.patch added
- Status changed from Nouveau to Solution proposée
Ça donne ça.
Updated by Benjamin Dauvergne over 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters over 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
tox: move pylint to separate environment (#64666)