Project

General

Profile

Développement #61397

Builds jenkins, lancer pylint même si les tests ne passent pas

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

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
03 February 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No
Club:
No

Description

Tel que c'est actuellement ça force à :
  1. Pousser son patch
  2. Voir que le build est rouge
  3. Corriger l'intégralité des erreurs rapportées
  4. Pousser son patch
  5. Voir que le build est toujours rouge parce que pylint

J'aimerais réduire le nombre d'étapes de ce cycle, en faisant en sorte que l'étape 3. rapporte les erreurs pylint également.

La cause c'est qu'on donne à tox une liste de commandes, genre pour wcs :

commands =
    py.test -v {env:COVERAGE:} --junitxml=junit-{envname}.xml {posargs:tests/}
    pylint: ./pylint.sh wcs/ tests/

Et si pytest plante pylint n'est pas exécuté.

Authentic est la seule brique où il n'y a pas ce problème, parce qu'il y a un environnement code-style qui s'exécute indépendamment. Est-ce qu'on ne ferait pas pareil partout ?


Related issues

Related to Combo - Développement #61493: tox, exécuter pylint indépendamment des testsFermé07 February 2022

Actions
Related to Chrono - Développement #63008: tox, exécuter pylint indépendamment des testsFermé21 March 2022

Actions
Related to w.c.s. - Développement #64666: tox, exécuter pylint indépendamment des testsFermé28 April 2022

Actions

History

#1

Updated by Thomas Noël almost 3 years ago

Valentin Deniaud a écrit :

il y a un environnement code-style qui s'exécute indépendamment. Est-ce qu'on ne ferait pas pareil partout ?

Perso l'idée me va bien, tu nous fais une proposition pour combo (et éventuellement wcs) ?

#3

Updated by Valentin Deniaud almost 3 years ago

  • Assignee set to Valentin Deniaud
#4

Updated by Valentin Deniaud almost 3 years ago

#5

Updated by Valentin Deniaud over 2 years ago

#6

Updated by Valentin Deniaud over 2 years ago

#7

Updated by Valentin Deniaud over 2 years ago

  • Status changed from Nouveau to Solution déployée

Voilà c'est fait toupar.

#8

Updated by Transition automatique over 2 years ago

Automatic expiration

Also available in: Atom PDF