Development #83824
passer à pylint 3.0
0%
Description
Il y a eu #81905 pour poser la limite de version en raison d'un retard de pylint-django, désormais résolu.
À priori il faut juste revert les patches partout, puis éventuellement corriger les nouvelles erreurs.
(mais perso jamais compris pourquoi on vivait avec pylint sans limite de version, ce qui casse régulièrement nos builds sans prévenir quand un nouveau warning est introduit)
Demandes liées
Historique
Mis à jour par Valentin Deniaud il y a 5 mois
- Lié à Development #82169: Passer à pylint 3.0 ajouté
Mis à jour par Frédéric Péters il y a 5 mois
(mais perso jamais compris pourquoi on vivait avec pylint sans limite de version, ce qui casse régulièrement nos builds sans prévenir quand un nouveau warning est introduit)
Perso, 1/ je crains que si on mettait une limite de version on ne passerait jamais sur les nouvelles versions (on vit ça à d'autres endroits, genre #82024); 2/ ça évite un moment de grand saut, qui pourrait amener à devoir corriger plein de choses.
Mis à jour par Paul Marillonnet il y a 5 mois
Juste par curiosité, est-ce qu’à un moment il y a eu le débat de, quitte à tout chambouler, une montée de version majeure de pylint VS une migration vers ruff ?
Mis à jour par Frédéric Péters il y a 5 mois
À ma connaissance on n'a jamais discuté de ruff (qui reste très jeune).
Mis à jour par Valentin Deniaud il y a 5 mois
Frédéric Péters a écrit :
Perso, 1/ je crains que si on mettait une limite de version on ne passerait jamais sur les nouvelles versions (on vit ça à d'autres endroits, genre #82024);
J'aurais plutôt fait une analogie avec les pre-commit hooks qu'avec une dépendance du code. Ils sont aussi dans une version fixée et on passe bien les mettre à jour de temps en temps.
2/ ça évite un moment de grand saut, qui pourrait amener à devoir corriger plein de choses.
Entre un grand saut à tête reposée et des mini sauts gérés dans une relative urgence...
Mis à jour par Valentin Deniaud il y a 5 mois
Paul Marillonnet a écrit :
quitte à tout chambouler, une montée de version majeure de pylint
Elle n'a de majeur que le nom, aucun chamboulement à prévoir (0 modif nécessaire au code de chrono, en tout cas).
Mis à jour par Frédéric Péters il y a 5 mois
Ils sont aussi dans une version fixée
Mais ça n'y est pas notre choix. ("pre-commit configuration aims to give a repeatable and fast experience and therefore intentionally doesn't provide facilities for "unpinned latest version" for hook repositories." -- https://pre-commit.com/#using-the-latest-version-for-a-repository)
Mis à jour par Valentin Deniaud il y a 5 mois
Dac, c'est bien que les raisons de la situation actuelle soient explicitées, restons comme ça.
Mis à jour par Paul Marillonnet il y a 5 mois
Valentin Deniaud a écrit :
Paul Marillonnet a écrit :
quitte à tout chambouler, une montée de version majeure de pylint
Elle n'a de majeur que le nom, aucun chamboulement à prévoir (0 modif nécessaire au code de chrono, en tout cas).
Ok top.
Mis à jour par Valentin Deniaud il y a 5 mois
- Lié à Development #84019: tox.ini: retirer la constrainte pylint<3 ajouté