Project

General

Profile

Development #50927

intégrer l'exécution de black à tox/jenkins

Added by Frédéric Péters over 3 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
05 February 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour assurer qu'un oubli ou une mauvaise configuration locale ne vienne perturber l'ordre des choses.


Files

0001-tox-add-black-via-pre-commit-to-tests.patch (971 Bytes) 0001-tox-add-black-via-pre-commit-to-tests.patch Frédéric Péters, 05 February 2021 05:35 PM
0002-jenkins-run-tox-black-part.patch (1.16 KB) 0002-jenkins-run-tox-black-part.patch Frédéric Péters, 05 February 2021 05:35 PM

Associated revisions

Revision f94c4647 (diff)
Added by Frédéric Péters over 3 years ago

tox: add black (via pre-commit) to tests (#50927)

Revision 7d29f064 (diff)
Added by Frédéric Péters over 3 years ago

jenkins: run tox black part (#50927)

History

#1

Updated by Frédéric Péters over 3 years ago

0001 pour modifier tox, 0002 pour modifier le Jenkinsfile parce que dans w.c.s. on y cite explicitement l'environnement pour tourner en parallèle django 1.11 et 2.2.

Ça passe par pre-commit pour gérer l'exécution etc. ça permet de garder le paramétrage centralisé.

Aussi, ça aurait peut-être pu être dans une étape jenkins explicite en plus mais avec trop de colonnes dans le tableau central d'un job jenkins le fait passer dessous, c'est moche etc.

C'est testé dans https://jenkins.entrouvert.org/job/wcs-wip/job/wip%252Fjenkins-check-black/ (avec le build 3 qui intègre un commit qui casse les choses).

#2

Updated by Frédéric Péters over 3 years ago

(une fois validé ici j'adapterai la chose pour les autres modules convertis à black).

#3

Updated by Emmanuel Cazenave over 3 years ago

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

Updated by Frédéric Péters over 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 7d29f06442d8601e258134172dc2b6f02b4f6d27
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Feb 5 16:21:03 2021 +0100

    jenkins: run tox black part (#50927)

commit f94c4647b58889db3d23ed00cb135c021e303657
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Feb 5 15:48:46 2021 +0100

    tox: add black (via pre-commit) to tests (#50927)
#5

Updated by Frédéric Péters over 3 years ago

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

Also available in: Atom PDF