Projet

Général

Profil

Development #61397

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

Ajouté par Valentin Deniaud il y a plus de 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
03 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non
Club:
Non

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 ?


Demandes liées

Lié à Combo - Development #61493: tox, exécuter pylint indépendamment des testsFermé07 février 2022

Actions
Lié à Chrono - Development #63008: tox, exécuter pylint indépendamment des testsFermé21 mars 2022

Actions
Lié à w.c.s. - Development #64666: tox, exécuter pylint indépendamment des testsFermé28 avril 2022

Actions

Historique

#1

Mis à jour par Thomas Noël il y a plus de 2 ans

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

Mis à jour par Valentin Deniaud il y a environ 2 ans

  • Assigné à mis à Valentin Deniaud
#4

Mis à jour par Valentin Deniaud il y a environ 2 ans

#5

Mis à jour par Valentin Deniaud il y a environ 2 ans

#6

Mis à jour par Valentin Deniaud il y a environ 2 ans

#7

Mis à jour par Valentin Deniaud il y a presque 2 ans

  • Statut changé de Nouveau à Solution déployée

Voilà c'est fait toupar.

#8

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF