Skip to content

KHAZZANI-Badr-TP-Tri#39

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

KHAZZANI-Badr-TP-Tri#39
BadMisterH wants to merge 2 commits intobcalou:mainfrom
BadMisterH:main

Conversation

@BadMisterH
Copy link
Copy Markdown

No description provided.

@bcalou
Copy link
Copy Markdown
Owner

bcalou commented Dec 5, 2023

Les conventions pep8 ne sont pas respectées.
Les tests ne passent pas (à cause de l'import de range qui devrait être sort.range).
Il n'y a pas de réponses aux questions dans le README, mais heureusement il y a les graphs demandés.
Pourtant, à quelques détails près, le code est bien écrit ! Mais tu ne peux pas rendre un TP qui ne répond pas à la base : le formattage, les tests et les questions.
Attention pour la suite, je compte sur toi car je vois bien que le code est au rendez-vous.
PS : on a le droit de copier/coller des bouts de code, mais il ne faut pas laisser des commentaires du genre "Assuming you have a module named 'range_array' for generating arrays"...

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