entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

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
Conversation 0 Commits 1 Files Changed 4 +4 -4
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
check-migrations.sh
View File

@ -12,7 +12,7 @@ DATABASES = {
}
}
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"]))')
vdeniaud commented 2023-11-30 18:15:45 +01:00
Outdated
Review
Copy Link

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
bdauvergne commented 2023-12-13 22:00:09 +01:00
Outdated
Review
Copy Link

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.
vdeniaud commented 2023-12-14 10:58:52 +01:00
Outdated
Review
Copy Link

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
TEMPFILE=`mktemp`
trap "rm -f ${TEMPFILE} ${CHECK_MIGRATIONS_SETTINGS}" EXIT

0
src/authentic2/apps.py → src/authentic2/app.py
View File

2
src/authentic2/settings.py
View File

@ -160,7 +160,7 @@ INSTALLED_APPS = (
'authentic2.apps.authenticators',
'authentic2.apps.journal',
'authentic2.backends',
'authentic2',
'authentic2.app.Authentic2Config',
'django_rbac',
'authentic2.a2_rbac',
'gadjo',

4
tox.ini
View File

@ -108,8 +108,8 @@ setenv =
SETUPTOOLS_USE_DISTUTILS=stdlib
NUMPROCESSES={env:NUMPROCESSES:1}
deps =
pylint<3
astroid<3
pylint
astroid
pylint-django
psycopg2-binary
django_webtest