Skip to content

KHAZZANIAffichage#41

Open
BadMisterH wants to merge 2 commits intobcalou:mainfrom
BadMisterH:main
Open

KHAZZANIAffichage#41
BadMisterH wants to merge 2 commits intobcalou:mainfrom
BadMisterH:main

Conversation

@BadMisterH
Copy link
Copy Markdown

No description provided.

@BadMisterH BadMisterH marked this pull request as ready for review November 19, 2023 14:18
@bcalou
Copy link
Copy Markdown
Owner

bcalou commented Nov 24, 2023

Des erreurs de formattage (pycodestyle) et le test algorithm ne passe pas, ce qui est essentiel. Tu as notamment divisé par 3 dans le calcul du jour ancre (c'est par 2).
Le fichier main fait sa propre vérification de date (au lieu d'utiliser la fonction de date.py) et ne répond pas à l'utilisateur sur le jour, ce qui était tout le but.
Heureusement le reste du code est soigné, mais attention poiur la suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants