lpoaura/weatherlink2pg

Code quality

Closed this issue · 2 comments

lpofredc commented

Il faudrait ajouter quelques outils de contrôle de la qualité du code au projet:

  • black
  • isort
  • flake8 & flake8-isort
  • pylint
  • pre-commit

Ce ne sont pas des dépendances de prod, a ajouter au group dev qui ne seront pas requis au programme:

poetry add --group=dev black isort flake8 flake8-isort pylint pre-commit

L'ajout de pre-commit pour executer systématiquement ces taches à chaque commit serait aussi pas mal, ça évite d'avoir à y penser et de passer du temps à configurer sont environnement de dev. ça exécute les différents check et reformatage à chaque commit.

On verra plus tard pour des workflow GitHub qui sont très pratiques ;)

JohanRocheteau commented

Bonjour,

  • J'ai ajouté les différents outils pour la vérification de la qualité du code.
  • J'avais tout codé en camelCase ayant appris de cette façon là, mais de toute évidence il vaut mieux utiliser le snake_case.
  • J'ai cherché un bon moment mais je n'ai pas trouvé d'autres solutions que de mettre une exception sur l'import des librairies, pylint me mettant une erreur à chaque fois même si je suis bien dans mon env Poetry.
lpofredc commented

Chouette, merci!