Permettre l'utilisation de pylint>3 (#84019) #189

Merged
bdauvergne merged 1 commits from wip/84019-tox-ini-retirer-la-constrainte-p into main 2023-12-18 08:09:44 +01:00
Owner

authentic2.apps module had to be renamed in authntic2.app and le
AppConfig class referenced explicitely in settings.py to prevent
collision with the authentic2.apps package (it only bothers pylint,
python3 is happy to load Django applications from authentic2.apps in the
previous situation).

authentic2.apps module had to be renamed in authntic2.app and le AppConfig class referenced explicitely in settings.py to prevent collision with the authentic2.apps package (it only bothers pylint, python3 is happy to load Django applications from authentic2.apps in the previous situation).
bdauvergne force-pushed wip/84019-tox-ini-retirer-la-constrainte-p from 72abcc66aa to 266da4e795 2023-11-28 13:37:00 +01:00 Compare
bdauvergne force-pushed wip/84019-tox-ini-retirer-la-constrainte-p from 266da4e795 to e37fcd98da 2023-11-28 14:47:31 +01:00 Compare
vdeniaud reviewed 2023-11-30 18:15:45 +01:00
@ -13,3 +13,3 @@
}
EOF
APPS=$(env DJANGO_SETTINGS_MODULE=authentic2.settings python -c 'from django.conf import settings; print(" ".join(p for p in (p.split(".")[-1] for p in settings.INSTALLED_APPS if not p.startswith("django.")) if p not in ["admin", "auth", "contenttypes"]))')
APPS=$(env DJANGO_SETTINGS_MODULE=authentic2.settings python -c 'from django.conf import settings; print(" ".join(p for p in ((p.split(".")[-1] if not p.endswith("Config") else p.split(".")[-3]) for p in settings.INSTALLED_APPS if not p.startswith("django.")) if p not in ["admin", "auth", "contenttypes"]))')
Owner

Si ça n'embête que pylint il faut préférer un # ignore plutôt que la modification d'un truc fragile IMO

Si ça n'embête que pylint il faut préférer un `# ignore` plutôt que la modification d'un truc fragile IMO
Author
Owner

Ce n'est pas possible, pylint n'arrive pas à résoudre tout les packages dans authentic2.apps, ce n'est pas juste une ligne qui l'embête.

Ce n'est pas possible, pylint n'arrive pas à résoudre tout les packages dans authentic2.apps, ce n'est pas juste une ligne qui l'embête.
Owner

Marrant en première lecture j'avais cru qu'on touchait ici à un script de config du paquet debian ou autre, clairement osef de ce qui se passe dans check-migrations.sh, déso pour le bruit

Marrant en première lecture j'avais cru qu'on touchait ici à un script de config du paquet debian ou autre, clairement osef de ce qui se passe dans check-migrations.sh, déso pour le bruit
bdauvergne changed title from Permettre l'utilisation de pylint>3 (#83019) to Permettre l'utilisation de pylint>3 (#84019) 2023-12-13 22:00:49 +01:00
vdeniaud approved these changes 2023-12-14 10:58:58 +01:00
bdauvergne force-pushed wip/84019-tox-ini-retirer-la-constrainte-p from e37fcd98da to 8eea700e99 2023-12-18 08:09:33 +01:00 Compare
bdauvergne merged commit 8eea700e99 into main 2023-12-18 08:09:44 +01:00
bdauvergne deleted branch wip/84019-tox-ini-retirer-la-constrainte-p 2023-12-18 08:09:44 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#189
No description provided.