Skip to content

Conversation

@Mesabloo
Copy link
Collaborator

No description provided.

@Mesabloo Mesabloo marked this pull request as ready for review October 16, 2024 16:12
Copy link
Owner

@smelc smelc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mesabloo> le sujet est bien mais je te propose de le simplifier afin que les étudiants ne se perdent pas dans l'implémentation de parseurs compliqués (expressions parenthésées avec gestion de la priorité) qui auront peu de valeur pédagogique.

Je préfère qu'ils se concentrent sur l'écriture des types et du code monadique (parseur + interpréteur).

Du coup j'ai fait des suggestions en ce sens. Peux-tu adapter le sujet en conséquence ?

@Mesabloo
Copy link
Collaborator Author

J'ai retiré l'instruction switch ainsi que les expressions composées.
Dommage parce que du coup y'a plus vraiment d'"erreur" possible lors de l'interprétation (hors bonus).

J'ai aussi rendu le else du if optionnel, parce que pourquoi pas.

@smelc
Copy link
Owner

smelc commented Oct 17, 2024

J'ai retiré l'instruction switch ainsi que les expressions composées.
Dommage parce que du coup y'a plus vraiment d'"erreur" possible lors de l'interprétation (hors bonus).

Il y a toujours le cas où la condition du if ne s'évalue pas vers un booléen 🙂

J'ai aussi rendu le else du if optionnel, parce que pourquoi pas.

Bonne idée 👍

@smelc
Copy link
Owner

smelc commented Oct 25, 2024

@Mesabloo> can you put the final touch so that we merge it and this officially becomes this year's project?

@Mesabloo
Copy link
Collaborator Author

Oups, c'est fait !

@smelc smelc merged commit 4ae504b into main Oct 28, 2024
1 check passed
@smelc smelc deleted the project-2024 branch October 28, 2024 16:35
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.

4 participants