Projet

Général

Profil

Development #67057

Use pylint --jobs flags to speed up linting step

Ajouté par A. Berriot il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
A. Berriot
Version cible:
-
Début:
06 juillet 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

pylint offers a --jobs flag to use multiple processes:

--jobs <n-processes>, -j <n-processes>
                        Use multiple processes to speed up Pylint. Specifying 0
                        will auto-detect the number of processors available to
                        use, and will cap the count on Windows to avoid hangs.
                        (default: 1)

This can speed up our CI steps even more

Fichiers

0001-ci-speed-up-CI-using-multiple-processes-with-pylint-.patch (1005 octets) 0001-ci-speed-up-CI-using-multiple-processes-with-pylint-.patch A. Berriot, 06 juillet 2022 10:27
0001-ci-speed-up-CI-using-multiple-processes-with-pylint-.patch (1005 octets) 0001-ci-speed-up-CI-using-multiple-processes-with-pylint-.patch A. Berriot, 06 juillet 2022 11:12

Demandes liées

Copié depuis Chrono - Development #67056: Use pylint --jobs flags to speed up linting stepFermé06 juillet 2022

Actions

Révisions associées

Révision 1b2560b2 (diff)
Ajouté par A. Berriot il y a presque 2 ans

ci: speed up CI using multiple processes with pylint (#67057)

Historique

#1

Mis à jour par A. Berriot il y a presque 2 ans

  • Copié depuis Development #67056: Use pylint --jobs flags to speed up linting step ajouté
#2

Mis à jour par A. Berriot il y a presque 2 ans

  • Fichier 0001-ci-speed-up-CI-using-multiple-processes-with-pylint-.patch supprimé
#4

Mis à jour par Frédéric Péters il y a presque 2 ans

Pas sûr que ça fonctionne sur wcs, il doit rester certaines dépendances sur l'ordre d'exécution des tests mais surtout on ne s'attend nulle part à avoir la db qui change de manière extérieure à l'exécution d'un test :

def test1():
    vide une table
    action
    vérif qu'il y a une seule ligne dans la table

def test2():
    pareil, même table
#5

Mis à jour par Frédéric Péters il y a presque 2 ans

(mais c'est juste pylint dont on parle ici faut m'oublier)

#7

Mis à jour par A. Berriot il y a presque 2 ans

Frédéric Péters a écrit :

(mais c'est juste pylint dont on parle ici faut m'oublier)

Oui, je vais faire un test avec xdist pour voir, mais ça m'étonnerait que ça fonctionne aussi simplement que pour les autres projets.

En tout cas là, la parallélisation pylint nous fait passer de 40m à 37m de build, c'est toujours ça de pris je pense

#8

Mis à jour par Frédéric Péters il y a presque 2 ans

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

Mis à jour par A. Berriot il y a presque 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 6976f792517e767695f5c95119863edb9c65e02a
Author: Agate <aberriot@entrouvert.com>
Date:   Wed Jul 6 10:27:30 2022 +0200

    ci: speed up CI using multiple processes with pylint (#67057)
#10

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

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF