MDL29/JacoBot

Fork 8

Resolves #70 : Add pre-commit test #71

Open

yannis-mlgrn wants to merge 6 commits from yannis-mlgrn/JacoBot:70_Add-pre-commit into main

yannis-mlgrn commented

2024-04-25 20:50:28 +02:00

Member

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 20:50:28 +02:00

yannis-mlgrn added 1 commit

2024-04-25 20:50:29 +02:00

✨ Add pre-commit tests 7baa091b00

yannis-mlgrn added 1 commit

2024-04-25 20:51:06 +02:00

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

HS-157 commented

2024-04-25 22:40:15 +02:00

Owner

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 10:17:19 +02:00

Author

Member

mybad en effet il y a une erreur

yannis-mlgrn added 1 commit

2024-04-26 10:26:10 +02:00

🐛 Fix pyproject.toml error 4821b84556

yannis-mlgrn commented

2024-04-26 10:27:14 +02:00

Author

Member

faut faire un poetry install --with dev

faut faire un `poetry install --with dev`

HS-157 commented

2024-04-26 21:55:32 +02:00

Owner

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 22:14:14 +02:00

yannis-mlgrn commented

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

Author

Member

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 11:50:41 +02:00

remove useless yaml file test 65c4706e68

yannis-mlgrn added 1 commit

2024-04-28 22:32:39 +02:00

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

yannis-mlgrn added 1 commit

2024-05-01 19:37:31 +02:00

update README.md b508da62f2