search

PnX-SI / GeoNature

Application de saisie et de synthèse des observations faune et flore
GNU General Public License v3.0
101 stars 102 forks source link

Lint Backend #3162

Open edelclaux opened 1 month ago

edelclaux commented 1 month ago

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:

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)

jacquesfize commented 21 hours ago

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 : https://github.com/PnX-SI/GeoNature/blob/2108176e5a00ad8b904bcb24de3d8786ffbdf4ba/.github/workflows/lint.yml#L13

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: https://github.com/PnX-SI/GeoNature/pull/2879