Development #61362
Pylint ne s'exécute pas
0%
Description
J’avais constaté dans les logs de build un
pylint run-test: commands[0] | /bin/bash -c './pylint.sh src/*/' Traceback (most recent call last): File "/tmp/jenkins-authentic2-auth-fedict-wip-wip%2F61045-tox-local-unit-tests-10/tox-jenkins/authentic2-auth-fedict/wip/61045-tox-local-unit-tests/pylint/lib/python3.9/site-packages/astroid/astpeephole.py", line 17, in <module> _TYPES = (_ast.Str, _ast.Bytes) AttributeError: module '_ast' has no attribute 'Str'à la suite de quoi j’avais tapé un
astroid!=2.5.7
dans le tox.ini, voyant cette restriction de version déjà posée dans ce même fichier pour w.c.s., et ai cru à tort que cela corrigeait le problème (en fait rien à voir, un git annotate
m’aurait appris que cela avait été posé dans w.c.s. pour des questions de perf).
Bref, le pylint.sh ne s’exécute pas, les erreurs pylint ne sont pas remontées.
Révisions associées
misc: remove unused imports (#61362)
misc: remove unused variable (#61362)
misc: add timeouts to requests (#61362)
misc: give imported class its own name (#61362)
misc: ignore modelform-uses-exclude (#61362)
misc: remove old import location fallback (#61362)
misc: use lazy logging (#61362)
misc: use with-statement to open files (#61362)
tox: do not run pylint on .egg-info directory (#61362)
tox: mark build in error if pylint fails (#61362)
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Sujet changé de jenkins : faux positif pylint.sh à Pylint ne s'exécute pas
Il faudra également penser à le faire tourner sur les tests (actuellement il ne vise que src/).
Mis à jour par Robot Gitea il y a 12 mois
- Tracker changé de Bug à Development
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic2-auth-fedict/pulls/5
- Titre : WIP: tox: remove pylint limits (#61362)
- Modifications : https://git.entrouvert.org/entrouvert/authentic2-auth-fedict/pulls/5/files
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution proposée à Solution validée
Paul Marillonnet (pmarillonnet) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic2-auth-fedict/pulls/5
- Titre : tox: remove pylint limits (#61362)
- Modifications : https://git.entrouvert.org/entrouvert/authentic2-auth-fedict/pulls/5/files
Mis à jour par Frédéric Péters il y a 6 mois
- Statut changé de Résolu (à déployer) à Solution déployée
tox: make pylint run (remove limits, add lasso and psycopg2) (#61362)