Development #53406
nouvelle notice pylint, consider-using-with
Début:
25 avril 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Nouvelle version de pylint et cette notice (catégorie refactoring), avec 103 occurences, sur 40 fichiers (surtout dans les tests), résultat pylint ici : https://jenkins.entrouvert.org/job/wcs/2561/pylint/new/type.-749254397/
(j'ai posé consider-using-with dans les tests désactivés pour le moment)
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Fichier 0001-misc-fix-pylint-consider-using-with-53406.patch 0001-misc-fix-pylint-consider-using-with-53406.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
J'ai survolé la partie tests, rien vu de problématique.
Pour le reste tout me semble bon aussi, à par peut-être un « return redirect('themes') » qui pourrait être factorisé sur la fonction install_theme_from_file (dans wcs/admin/settings.py) mais bon c'est du total pinaillage.
Ok pour envoyer ça rapidement en recette que ça tourne 2 semaines.
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit b6e849811ac3990a366f22dffb7415e03c0b4a64 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Thu Apr 29 18:11:24 2021 +0200 misc: fix pylint consider-using-with (#53406)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: fix pylint consider-using-with (#53406)