Development #52222
pylint - réduire les warnings
0%
Description
https://jenkins.entrouvert.org/job/wcs/2445/pylint/
nettoyer tout ça, histoire de voir les vrais soucis; exemple: https://git.entrouvert.org/wcs.git/commit/?id=af87cabb0d9ead7c99dc94dfebb5d715ab09d3d9
Fichiers
Révisions associées
misc: pylint fix wrong-import-position (#52222)
misc: pylint fix unused-wildcard-import (#52222)
misc: pylint fix super-with-arguments (#52222)
misc: pylint fix unused-import (#52222)
misc: pylint fix unused-variable (#52222)
misc: pylint fix bare-except (#52222)
misc: pylint fix spacing around comments (#52222)
misc: pylint fix dangerous-default-value (#52222)
misc: pylint fix useless-object-inheritance (#52222)
misc: pylint fix unidiomatic-typecheck (#52222)
misc: pylint fix anomalous-backslash-in-string (#52222)
misc: pylint fix simplifiable-if-statement (#52222)
misc: pylint fix function-redefined (#52222)
misc: pylint fix bad-classmethod-argument (#52222)
misc: pylint fix no-else-raise (#52222)
misc: pylint fix consider-using-in (#52222)
misc: pylint fix unneeded-not (#52222)
misc: pylint fix using-constant-test (#52222)
misc: pylint fix chained-comparison (#52222)
misc: pylint fix import-error (#52222)
misc: pylint fix consider-merging-isinstance (#52222)
misc: pylint fix ungrouped-imports (#52222)
misc: pylint fix reimported (#52222)
misc: pylint fix no-else-continu (#52222)
misc: pylint fix no-else-break (#52222)
misc: pylint fix useless-else-on-loop (#52222)
misc: pylint fix wrong placeholders & substitutions (#52222)
misc: pylint fix access-member-before-definition (#52222)
misc: pylint fix unnecessary-pass (#52222)
misc: pylint fix no-self-argument (#52222)
misc: remove dead code (#52222)
misc: pylint fix last warnings (#52222)
Historique
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0010-misc-pylint-fix-useless-object-inheritance-52222.patch ajouté
- Fichier 0009-misc-pylint-fix-dangerous-default-value-52222.patch 0009-misc-pylint-fix-dangerous-default-value-52222.patch ajouté
- Fichier 0008-misc-pylint-fix-spacing-around-comments-52222.patch 0008-misc-pylint-fix-spacing-around-comments-52222.patch ajouté
- Fichier 0007-misc-pylint-fix-bare-except-52222.patch 0007-misc-pylint-fix-bare-except-52222.patch ajouté
- Fichier 0006-misc-pylint-fix-unused-variable-52222.patch 0006-misc-pylint-fix-unused-variable-52222.patch ajouté
- Fichier 0005-misc-pylint-fix-unused-import-52222.patch 0005-misc-pylint-fix-unused-import-52222.patch ajouté
- Fichier 0004-misc-pylint-fix-super-with-arguments-52222.patch 0004-misc-pylint-fix-super-with-arguments-52222.patch ajouté
- Fichier 0003-misc-pylint-fix-unused-wildcard-import-52222.patch 0003-misc-pylint-fix-unused-wildcard-import-52222.patch ajouté
- Fichier 0002-misc-pylint-fix-wrong-import-position-52222.patch 0002-misc-pylint-fix-wrong-import-position-52222.patch ajouté
- Fichier 0001-misc-integrate-pylint.rc-in-repo-52222.patch 0001-misc-integrate-pylint.rc-in-repo-52222.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
note: pour le moment j'ai réintégré pylint.rc dans le repo
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0020-misc-pylint-fix-chained-comparison-52222.patch 0020-misc-pylint-fix-chained-comparison-52222.patch ajouté
- Fichier 0019-misc-pylint-fix-using-constant-test-52222.patch 0019-misc-pylint-fix-using-constant-test-52222.patch ajouté
- Fichier 0018-misc-pylint-fix-unneeded-not-52222.patch 0018-misc-pylint-fix-unneeded-not-52222.patch ajouté
- Fichier 0017-misc-pylint-fix-consider-using-in-52222.patch 0017-misc-pylint-fix-consider-using-in-52222.patch ajouté
- Fichier 0016-misc-pylint-fix-no-else-raise-52222.patch 0016-misc-pylint-fix-no-else-raise-52222.patch ajouté
- Fichier 0015-misc-pylint-fix-bad-classmethod-argument-52222.patch 0015-misc-pylint-fix-bad-classmethod-argument-52222.patch ajouté
- Fichier 0014-misc-pylint-fix-function-redefined-52222.patch 0014-misc-pylint-fix-function-redefined-52222.patch ajouté
- Fichier 0013-misc-pylint-fix-simplifiable-if-statement-52222.patch 0013-misc-pylint-fix-simplifiable-if-statement-52222.patch ajouté
- Fichier 0012-misc-pylint-fix-anomalous-backslash-in-string-52222.patch 0012-misc-pylint-fix-anomalous-backslash-in-string-52222.patch ajouté
- Fichier 0011-misc-pylint-fix-unidiomatic-typecheck-52222.patch ajouté
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0030-misc-pylint-fix-unnecessary-pass-52222.patch 0030-misc-pylint-fix-unnecessary-pass-52222.patch ajouté
- Fichier 0029-misc-pylint-fix-access-member-before-definition-5222.patch 0029-misc-pylint-fix-access-member-before-definition-5222.patch ajouté
- Fichier 0028-misc-pylint-fix-wrong-placeholders-substitutions-522.patch 0028-misc-pylint-fix-wrong-placeholders-substitutions-522.patch ajouté
- Fichier 0027-misc-pylint-fix-useless-else-on-loop-52222.patch 0027-misc-pylint-fix-useless-else-on-loop-52222.patch ajouté
- Fichier 0026-misc-pylint-fix-no-else-break-52222.patch 0026-misc-pylint-fix-no-else-break-52222.patch ajouté
- Fichier 0025-misc-pylint-fix-no-else-continu-52222.patch 0025-misc-pylint-fix-no-else-continu-52222.patch ajouté
- Fichier 0024-misc-pylint-fix-reimported-52222.patch 0024-misc-pylint-fix-reimported-52222.patch ajouté
- Fichier 0023-misc-pylint-fix-ungrouped-imports-52222.patch 0023-misc-pylint-fix-ungrouped-imports-52222.patch ajouté
- Fichier 0022-misc-pylint-fix-consider-merging-isinstance-52222.patch 0022-misc-pylint-fix-consider-merging-isinstance-52222.patch ajouté
- Fichier 0021-misc-pylint-fix-import-error-52222.patch 0021-misc-pylint-fix-import-error-52222.patch ajouté
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0033-misc-pylint-fix-last-warnings-52222.patch ajouté
- Fichier 0032-misc-remove-dead-code-52222.patch 0032-misc-remove-dead-code-52222.patch ajouté
- Fichier 0031-misc-pylint-fix-no-self-argument-52222.patch 0031-misc-pylint-fix-no-self-argument-52222.patch ajouté
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier
0011-misc-pylint-fix-unidiomatic-typecheck-52222.patchsupprimé
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier
0010-misc-pylint-fix-useless-object-inheritance-52222.patchsupprimé
Mis à jour par Lauréline Guérin il y a environ 3 ans
Mis à jour par Lauréline Guérin il y a environ 3 ans
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier
0033-misc-pylint-fix-last-warnings-52222.patchsupprimé
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Je pioche juste quelques patchs au hasard, toute la partie form.py dans 0014-misc-pylint-fix-function-redefined-52222.patch est super, ça correspond à ce vieux projet #5790, ça fait partie des trucs non-triviaux que je n'imaginais pas ici.
Je dirais que c'est ok pour moi, sauf si tu souhaites que je regarde particulièrement un commit particulier.
Mis à jour par Lauréline Guérin il y a environ 3 ans
dans 0003, j'ai simplifié des trucs dans les tests autour de PicklableUpload (des imports dans les tests en PicklableUpload2 ou PicklableUpload3), liés à #13777, et je suis pas bien sûre de moi
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Sujet changé de pytlint - réduire les warnings à pylint - réduire les warnings
Mis à jour par Frédéric Péters il y a environ 3 ans
Je pense que c'était juste un problème avec Python 2, c'est ok pour la suppression.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6632a3133bd1cd18e71a2de0c9ec1cc31eb581a3 (HEAD -> main, origin/wip/52222-pylint, origin/main, wip/52222-pylint) Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 16:45:18 2021 +0100 misc: pylint fix last warnings (#52222) commit 4889deb2a0973234981cd23489ff61d608701b62 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 16:37:39 2021 +0100 misc: remove dead code (#52222) commit 87e806a8940169673b8f5c39bd9246713ff3a255 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:25:26 2021 +0100 misc: pylint fix no-self-argument (#52222) commit ceee15b3a136cb8f413926040f213fe239ed37ec Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:24:06 2021 +0100 misc: pylint fix unnecessary-pass (#52222) commit 9d7692cbab6c46bd6e272cb742d8ee0eec58e74d Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:23:11 2021 +0100 misc: pylint fix access-member-before-definition (#52222) commit 6ac7eb4a43cce83760203e1c391c732122910afc Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:18:13 2021 +0100 misc: pylint fix wrong placeholders & substitutions (#52222) commit 3c2ddb1f2295dcf69c26fb74479c38b071e7b441 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:11:52 2021 +0100 misc: pylint fix useless-else-on-loop (#52222) commit cec0ad3087b1bf625c24d9c154630a482eb61828 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:09:19 2021 +0100 misc: pylint fix no-else-break (#52222) commit bcc6a9bc2fad64d9901459189a0df1c9de9bd784 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 15:05:58 2021 +0100 misc: pylint fix no-else-continu (#52222) commit 39034b7de16fd28350a59a404c240022149c8826 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 14:59:39 2021 +0100 misc: pylint fix reimported (#52222) commit a7b8150b26ba1998fbc5061a0d7ec6d0154263fc Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 14:56:13 2021 +0100 misc: pylint fix ungrouped-imports (#52222) commit 0c856c719a709e2b7a94ee3dd7b8cca43502ceb6 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 14:26:44 2021 +0100 misc: pylint fix consider-merging-isinstance (#52222) commit 191013095583ba6c1dbe0dc140772b751dcde321 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 11:53:54 2021 +0100 misc: pylint fix import-error (#52222) commit 101f8d06b2bc97d6e5e1c288e747747a82f84b3b Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 11:33:58 2021 +0100 misc: pylint fix chained-comparison (#52222) commit 021b73d4fb59db388a17569632f0b6c4d6f25d35 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 11:27:05 2021 +0100 misc: pylint fix using-constant-test (#52222) commit d88dd530f61557c0c06e8dae55f7689af34a4621 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 11:14:42 2021 +0100 misc: pylint fix unneeded-not (#52222) commit 17d73057b718db60fc60a7b0417aa533a7ffb719 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 11:02:15 2021 +0100 misc: pylint fix consider-using-in (#52222) commit f2a3f4c3003b892beeda77be7a4bb504727f2814 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 10:59:34 2021 +0100 misc: pylint fix no-else-raise (#52222) commit daf108ffd546223029dd5ada7f4568459c8a6e4e Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 10:56:28 2021 +0100 misc: pylint fix bad-classmethod-argument (#52222) commit d3f8af1166ae85557ab9e94441eed6c671645377 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 10:28:40 2021 +0100 misc: pylint fix function-redefined (#52222) commit 46be3ca3ceca97bb50f02fddaac2773ffff7906e Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 10:09:32 2021 +0100 misc: pylint fix simplifiable-if-statement (#52222) commit e8ec57ac999d58df9699d409acc1b57858819c3a Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 09:43:39 2021 +0100 misc: pylint fix anomalous-backslash-in-string (#52222) commit 9b448b3232bd6021696ef88c469c39fbceb41311 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 09:23:31 2021 +0100 misc: pylint fix unidiomatic-typecheck (#52222) commit f21431cb8a8db1cada58899cc1451de749631cb6 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 09:19:23 2021 +0100 misc: pylint fix useless-object-inheritance (#52222) commit edab61d2270a800203fb27c4b704de9ae24c2cdc Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Sat Mar 20 23:46:43 2021 +0100 misc: pylint fix dangerous-default-value (#52222) commit 14c491d93b298845ed58de6fe3c9a217ec539cba Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Sat Mar 20 23:25:10 2021 +0100 misc: pylint fix spacing around comments (#52222) commit 41bf7dfac94e1716a6d674c10d956e7d3791fe58 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Sat Mar 20 23:07:57 2021 +0100 misc: pylint fix bare-except (#52222) commit 157f97a27a556c3e6d41745d7933324ed1c9b30d Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 22:49:33 2021 +0100 misc: pylint fix unused-variable (#52222) commit fe8b9f3d7a591ae302f23a40d0c79d27be80f323 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 18:48:46 2021 +0100 misc: pylint fix unused-import (#52222) commit 6893e0dd80d7307cd0c86f0b44e97e372d9ae33b Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 18:39:01 2021 +0100 misc: pylint fix super-with-arguments (#52222) commit 6ca8dc014c52dd221a2454e6f03431d4d5a03a21 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 16:53:37 2021 +0100 misc: pylint fix unused-wildcard-import (#52222) commit 04f68506dc56e7370b75416cc78f5f478999719d Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Mar 19 14:25:04 2021 +0100 misc: pylint fix wrong-import-position (#52222) commit 97f7e9d51622af6eb123c7dda54cf188a07a7364 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Mar 22 14:44:53 2021 +0100 misc: integrate pylint.rc in repo (#52222)
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: integrate pylint.rc in repo (#52222)