Resolves #70 : Add pre-commit test #71

yannis-mlgrn · 2024-04-25T18:50:28Z

yannis-mlgrn commented

2024-04-25 18:50:28 +00:00

Resolves #70

I've added pre-commit to run test before every commits to be sure that littles petits hackers respect the PEP8 convention 😆

- Resolves #70 I've added pre-commit to run test before every commits to be sure that littles petits hackers respect the PEP8 convention 😆

yannis-mlgrn added the

Feature

JacoVirt

labels 2024-04-25 18:50:28 +00:00

yannis-mlgrn added 1 commit 2024-04-25 18:50:29 +00:00

7baa091b00

✨ Add pre-commit tests

yannis-mlgrn added 1 commit 2024-04-25 18:51:06 +00:00

6ae7653934

Merge remote-tracking branch 'upstream/main' into 70_Add-pre-commit

HS-157 commented

2024-04-25 20:40:15 +00:00

Je pense que tu n'as pas testé ce que tu as fais.

❯ poetry install

Cannot declare ('tool', 'poetry', 'group', 'dev', 'dependencies') twice (at line 18, column 36)

Et c'est quoi ce truc ? Ça me parait un truc trop complexe pour le besoin qu'on a et surtout, ça télécharge et installe des trucs automagiquement.

Je pense que tu n'as pas testé ce que tu as fais. ``` ❯ poetry install Cannot declare ('tool', 'poetry', 'group', 'dev', 'dependencies') twice (at line 18, column 36) ``` Et c'est quoi ce truc ? Ça me parait un truc trop complexe pour le besoin qu'on a et surtout, ça télécharge et installe des trucs automagiquement.

yannis-mlgrn commented

2024-04-26 08:17:19 +00:00

mybad en effet il y a une erreur

yannis-mlgrn added 1 commit 2024-04-26 08:26:10 +00:00

4821b84556

🐛 Fix pyproject.toml error

yannis-mlgrn commented

2024-04-26 08:27:14 +00:00

faut faire un poetry install --with dev

faut faire un `poetry install --with dev`

HS-157 commented

2024-04-26 19:55:32 +00:00

Tu n'as pas répondu à ma question, qu'est-ce que cet outil apporte ? Et comment l'utiliser ?

HS-157 self-assigned this 2024-04-26 20:14:14 +00:00

yannis-mlgrn commented

2024-04-27 09:42:02 +00:00

pre-commit sert lorsque tu commit des changements à lancer des tests.
J'ai ajouté comme tests :

black qui reformate le code proprement ("Black is a PEP 8 compliant opinionated formatte")
un test qui vérifie qu'il n'y a pas de clé privée dedans ( pour plus tard on sait jamais )
un test qui vérifie si tu ne commit pas des fichiers volumineux, si c'est le cas il va te demander de confirmer pour commit
un test qui verifie si tu n'est pas en conflit avec la branche main
et il run le linter (flake8) et si y'a des erreurs de linter il te les ressort et t'empêche de commit des modifications qui ne respecte pas PEP8

Tu peux néanmoins passer des tests avec SKIP=flake8 git commit -m "foo" par exemple

Mais ça me semble intéressant et ce qui va déjà filtrer quelques erreurs récurrentes, et ça se lance automatiquement lorsque tu commit quelque chose. Tu peux aussi configurer pour lancer des tests spécifiques lorsque tu push ou merge mais là j'au utilisé que lors d'un commit

pre-commit sert lorsque tu commit des changements à lancer des tests. J'ai ajouté comme tests : - black qui reformate le code proprement ("Black is a PEP 8 compliant opinionated formatte") - un test qui vérifie qu'il n'y a pas de clé privée dedans ( pour plus tard on sait jamais ) - un test qui vérifie si tu ne commit pas des fichiers volumineux, si c'est le cas il va te demander de confirmer pour commit - un test qui verifie si tu n'est pas en conflit avec la branche main - et il run le linter (flake8) et si y'a des erreurs de linter il te les ressort et t'empêche de commit des modifications qui ne respecte pas PEP8 Tu peux néanmoins passer des tests avec `SKIP=flake8 git commit -m "foo"` par exemple Mais ça me semble intéressant et ce qui va déjà filtrer quelques erreurs récurrentes, et ça se lance automatiquement lorsque tu commit quelque chose. Tu peux aussi configurer pour lancer des tests spécifiques lorsque tu push ou merge mais là j'au utilisé que lors d'un commit

yannis-mlgrn added 1 commit 2024-04-27 09:50:41 +00:00

65c4706e68

remove useless yaml file test

yannis-mlgrn added 1 commit 2024-04-28 20:32:39 +00:00

148aa63f47

Merge remote-tracking branch 'upstream/main' into 70_Add-pre-commit

yannis-mlgrn added 1 commit 2024-05-01 17:37:31 +00:00

b508da62f2

update README.md

yannis-mlgrn commented

2024-05-01 17:40:07 +00:00

normallement c'est bon, pour installer il y a juste une seule commande en plus.
preview :