Projet

Général

Profil

Development #50927

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

Ajouté par Frédéric Péters il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
05 février 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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

Fichiers

0001-tox-add-black-via-pre-commit-to-tests.patch (971 octets) 0001-tox-add-black-via-pre-commit-to-tests.patch Frédéric Péters, 05 février 2021 17:35
0002-jenkins-run-tox-black-part.patch (1,16 ko) 0002-jenkins-run-tox-black-part.patch Frédéric Péters, 05 février 2021 17:35

Révisions associées

Révision f94c4647 (diff)
Ajouté par Frédéric Péters il y a environ 3 ans

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

Révision 7d29f064 (diff)
Ajouté par Frédéric Péters il y a environ 3 ans

jenkins: run tox black part (#50927)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 3 ans

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

Mis à jour par Frédéric Péters il y a environ 3 ans

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

#3

Mis à jour par Emmanuel Cazenave il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée
#4

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Solution validée à 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

Mis à jour par Frédéric Péters il y a environ 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF