-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: update black parameter to meet line-length of 100 chars requirement #2847
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2847 +/- ##
========================================
Coverage 78.43% 78.43%
========================================
Files 88 88
Lines 7183 7183
========================================
Hits 5634 5634
Misses 1549 1549
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
c2fa832
to
b7527fc
Compare
Tu peux me donner l'autorisation de push sur ta branche ? J'ai fix le lint dans plusieurs fichiers du backend. |
Comme c'est un changement de la config du lint du backend, j'ai volontairement pas linté les fichiers. (Je viens de pousser la version avec un commit de lintage, qui se drope si besoin) |
Ah si tu as fait les modifs, c'est tout bon. Par contre, tu as fait une modification sur le module d'authentification ? |
non, effectivement. Je sais pas d'où est sortie cette modif. C'est corrigé |
Une erreur de ma part :s J'ai crée la branche par erreur et je dois la supprimer. [ C'est fait ! ] |
[Périmètre] lint python 99 vs 100
[Problème]
La doc stipule un line-length de 100, mais la valeur par défaut de black est fixée dans le projet à 99
Doc: https://docs.geonature.fr/development.html#backend
Project configuration
pyproject.toml
:[Solution]
Forcer le lint à la valeur stipulée dans la doc
Remarque --> si 100 n'est plus une valeur satisfaisante, on peut faire varier la valeur doc et le pyproject conjointement