Project

General

Profile

Développement #64666

tox, exécuter pylint indépendamment des tests

Added by Valentin Deniaud over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
28 April 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Comme fait ailleurs.


Files

0001-wip.patch (1.97 KB) 0001-wip.patch Valentin Deniaud, 09 May 2022 04:23 PM
0001-tox-move-pylint-to-separate-environment-64666.patch (1.76 KB) 0001-tox-move-pylint-to-separate-environment-64666.patch Valentin Deniaud, 11 May 2022 11:02 AM

Related issues

Related to Publik - Développement #61397: Builds jenkins, lancer pylint même si les tests ne passent pasFermé03 February 2022

Actions

Associated revisions

Revision 3a5b67b0 (diff)
Added by Valentin Deniaud over 2 years ago

tox: move pylint to separate environment (#64666)

History

#1

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
#2

Updated by Valentin Deniaud over 2 years ago

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.

#3

Updated by Valentin Deniaud over 2 years ago

  • Status changed from Solution proposée to Nouveau
#6

Updated by Valentin Deniaud over 2 years ago

Ça donne ça.

#7

Updated by Benjamin Dauvergne over 2 years ago

  • Status changed from Solution proposée to Solution validée
#8

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)
#9

Updated by Transition automatique over 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#10

Updated by Transition automatique over 2 years ago

Automatic expiration

Also available in: Atom PDF