Skip to content
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

Lint Backend #3162

Open
edelclaux opened this issue Aug 21, 2024 · 1 comment
Open

Lint Backend #3162

edelclaux opened this issue Aug 21, 2024 · 1 comment
Labels

Comments

@edelclaux
Copy link
Contributor

Version
Branche "develop"

Description du bug
Quelques fichiers du backend sont modifiés quand on fait tourner le linter avec la config par défaut.
J'ai des modif quand je fais tourner black 23.11:

	modifié :         backend/geonature/core/gn_commons/schemas.py
	modifié :         backend/geonature/core/gn_meta/routes.py
	modifié :         backend/geonature/tests/benchmarks/benchmark_generator.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_gn_meta.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_occhab.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_occtax.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_ref_geo.py
	modifié :         backend/geonature/tests/utils.py
	modifié :         backend/geonature/utils/celery.py

ou black 24.8 (latest):

	modifié :         backend/geonature/core/gn_commons/schemas.py
	modifié :         backend/geonature/tests/benchmarks/benchmark_generator.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_gn_meta.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_occhab.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_occtax.py
	modifié :         backend/geonature/tests/benchmarks/test_benchmark_ref_geo.py
	modifié :         backend/geonature/tests/utils.py
	modifié :         backend/geonature/utils/celery.py

Question
Est-ce que la config a changé ?
Quelle est la version de black de référence utilisée ?

Proposition
Est-ce que vous pensez que ce serait une bonne idée de contraindre cette version:

  • rajouter black dans les requirements dev
  • rajouter le numéro de version dans la githubactions du lint

Si oui, on s'appuierait sur la dernière, la 24.8 ?

Quelqu'un a une suggestion pour ne renseigner ce numéro de version qu'une seule fois (gh actions ET requirements dev)

@edelclaux edelclaux added the bug label Aug 21, 2024
@jacquesfize
Copy link
Contributor

jacquesfize commented Oct 9, 2024

Salut @edelclaux,

Pour le formatage avec black, on utilise la dernière version stable (la 24.10 normalement). On peut le voir dans la CI :

uses: psf/black@stable

Par contre, je suis d'accord, c'est une bonne idée de rajouter la dépendance black dans le requirement dev. C'est cohérent avec l'installation de prettier dans les dependances dev côté frontend.

PR de la maj du lint avec black 24: #2879

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants