Development #52484
pylint, référence aux tests désactivés en utilisant leur code, pas leur numéro
0%
Description
J'étais étonné de voir des traces de pylint dans les builds récents mais c'est apparemment parce que le pylint ne prends plus les numéros (ex: C0103) dans sa configuration des tests désactivés, il prend les codes plus lisibles (ex: abstract-method).
Ici la conversion pour tout prendre de ce qui existait, il y aurait un travail supplémentaire à faire pour ne pas reprendre ceux qui ne produisent aujourd'hui pas/plus.
Fichiers
Révisions associées
build: update pylint to disable errors by code, not number (#52484)
build: update pylint disable list to actually happening warnings (#52484)
misc: refer to pylint codes, not numbers (#52484)
Historique
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a environ 3 ans
- Fichier 0004-misc-refer-to-pylint-codes-not-numbers-52484.patch 0004-misc-refer-to-pylint-codes-not-numbers-52484.patch ajouté
- Fichier 0003-build-update-pylint-disable-list-to-actually-happeni.patch 0003-build-update-pylint-disable-list-to-actually-happeni.patch ajouté
- Fichier 0002-build-update-pylint-to-disable-errors-by-code-not-nu.patch 0002-build-update-pylint-to-disable-errors-by-code-not-nu.patch ajouté
- Fichier 0001-misc-remove-unused-imports-52484.patch 0001-misc-remove-unused-imports-52484.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Ça remplace donc les références aux erreurs des numéros aux codes, je pense possible que ça vienne d'une erreur d'interprétation de ma part au début, perdu dans les rapports jenkins, mais j'ai continué parce que je trouve quand même ça plus compréhensible.
Ensuite, à faire le tour, il y a un commit peu lisible parce que longue ligne mais ça retire bad-continuation, bad-whitespace, relative-import et too-many-format-args.
Il y a aussi dans la branche un premier commit qui corrige des commits introduits via #49405.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
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 feb75906356ba2dbff92cc8f3fe9d3923801c1f9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 30 11:13:29 2021 +0200
misc: refer to pylint codes, not numbers (#52484)
commit e8fbf9a60c652a7bff6958f540612df95e8db1d6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 30 09:11:11 2021 +0200
build: update pylint disable list to actually happening warnings (#52484)
commit 8f43b58ba1be5015fb52375f5ef53efeb8d24d16
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 30 08:29:04 2021 +0200
build: update pylint to disable errors by code, not number (#52484)
commit 0ba975973987200d61a204a1cafd07081dd56c3a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Mar 30 11:22:27 2021 +0200
misc: remove unused imports (#52484)
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
misc: remove unused imports (#52484)