HS-157 commented on pull request MDL29/JacoBot#66 2024-04-26 21:22:57 +00:00
Creates a garbage zone for the virtual pad #34

Cette ligne devrait être en dehors de la condition car, dans ce cas là, on a toujours le token dans la main tant qu'on n'a pas cliqué autre part que sur la poubelle ou qu'on a un autre token dans la main.

HS-157 opened issue MDL29/JacoBot#75 2024-04-26 21:08:36 +00:00
Add animation in JacoVirt Bot
HS-157 opened issue MDL29/JacoBot#74 2024-04-26 21:03:39 +00:00
Add game design element
HS-157 opened issue MDL29/JacoBot#73 2024-04-26 20:58:53 +00:00
Add a bot in JacoVirt Bot
HS-157 opened issue MDL29/JacoBot#72 2024-04-26 20:49:31 +00:00
Draw cells on JacoVirt Bot
HS-157 suggested changes for MDL29/JacoBot#66 2024-04-26 20:45:56 +00:00
Creates a garbage zone for the virtual pad #34
HS-157 commented on pull request MDL29/JacoBot#66 2024-04-26 20:45:56 +00:00
Creates a garbage zone for the virtual pad #34

Attention en faisant comme ça, ta ligne 118 devient doublon avec la ligne 127, il faut regarder un peu plus loin que son code quand on rajoute des modifications. 🙂

HS-157 commented on pull request MDL29/JacoBot#21 2024-04-26 20:13:31 +00:00
15: improve README: license, contributing.

Rajouter un lien vers le projet Cubetto.

HS-157 commented on pull request MDL29/JacoBot#60 2024-04-26 20:10:16 +00:00
Improve JacoVirt Bot window

Pour le moment, je ne vois pas l'utilité d'avoir un compteur pour mettre des ID aux cases.

HS-157 commented on pull request MDL29/JacoBot#60 2024-04-26 20:10:16 +00:00
Improve JacoVirt Bot window

Je ne vois pas où est dessiné cette forme et à quoi elle sert.

HS-157 commented on pull request MDL29/JacoBot#60 2024-04-26 20:10:16 +00:00
Improve JacoVirt Bot window

Je ne pense pas que cette ligne devrait être dans cette méthode, mais plus dans le __init__().

HS-157 commented on pull request MDL29/JacoBot#60 2024-04-26 20:10:16 +00:00
Improve JacoVirt Bot window

Mettre cette classe dans un autre fichier et je l'aurai comme nom Cell.

HS-157 suggested changes for MDL29/JacoBot#60 2024-04-26 20:10:16 +00:00
Improve JacoVirt Bot window
HS-157 commented on pull request MDL29/JacoBot#71 2024-04-26 19:55:32 +00:00
Resolves #70 : Add pre-commit test

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

HS-157 suggested changes for MDL29/JacoBot#66 2024-04-26 19:51:16 +00:00
Creates a garbage zone for the virtual pad #34

Ça plante quand on clique sur la poubelle quand on n'a pas de token dans la main.

HS-157 commented on pull request MDL29/JacoBot#71 2024-04-25 20:40:15 +00:00
Resolves #70 : Add pre-commit test

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…

HS-157 pushed to main at MDL29/JacoBot 2024-04-25 14:46:31 +00:00
041307a28c Add flake8 linter
HS-157 merged pull request MDL29/JacoBot#65 2024-04-25 14:46:29 +00:00
[ LINTER ] Add flake8
HS-157 approved MDL29/JacoBot#65 2024-04-25 14:44:11 +00:00
[ LINTER ] Add flake8
HS-157 approved MDL29/JacoBot#64 2024-04-25 14:10:33 +00:00
Improve README for JacoVirt