Development #67470
erreur pylint "invalid-str-returned"
0%
Description
def __str__(self): return self.title
J'imagine qu'il est mal à l'aise parce que title c'est models.CharField(...).
Je proposerais d'ajouter invalid-str-returned dans pylint.rc (parce que je suis surpris qu'il n'en trouve pas d'autres)
Sauf qu'à regarder il n'y a pas eu de nouvelle version pylint entre https://jenkins.entrouvert.org/job/combo/2796/ ok et https://jenkins.entrouvert.org/job/combo/2797/ pas ok, et vite fait je ne vois pas de différence.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Nouveau à Fermé
Et build relancé et là pylint n'indique plus rien…
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Fermé à Nouveau
Et pas de modifs et là sur deux builds d'affilée l'erreur…
Mis à jour par Frédéric Péters il y a presque 2 ans
- Fichier 0001-misc-wrap-Page-__str__-value-in-str-to-please-pylint.patch 0001-misc-wrap-Page-__str__-value-in-str-to-please-pylint.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Frédéric Péters
- Patch proposed changé de Non à Oui
Au troisième build c'est passé mais ça n'est pas une vie, patch pour taper le retour dans str(), ça devrait calmer de manière systématique pylint.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution proposée à Nouveau
Oui Lauréline avait tapé ça partout au moment d'ajouter pylint (000af39adf87c253239d2f845d93534cb3be6141), pareil dans d'autres briques, par contre dans chrono l'erreur fait parti de celles qu'on ignore.
Sauf que, si pylint détecte ça de manière non déterministe, le problème risque de se reposer régulièrement, genre patch -> build ok -> push -> un jour cette erreur. On serait pas plus tranquilles à juste l'ignorer ?
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Nouveau à Solution proposée
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4a8f18d89ba39801822230511452d3e97fcc9c35 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Jul 21 13:26:03 2022 +0200 misc: wrap Page __str__ value in str() to please pylint (#67470)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: wrap Page str value in str() to please pylint (#67470)