Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Ajout des booléens #13

Merged
merged 20 commits into from
May 17, 2024
Merged

Ajout des booléens #13

merged 20 commits into from
May 17, 2024

Conversation

ilo80
Copy link
Contributor

@ilo80 ilo80 commented May 7, 2024

Détail

  • Ajout du type booléens
  • Ajout de l'opération "=="
  • Ajout de l'opération "!="
  • Ajout de l'opération ">"
  • Ajout de l'opération "<"
  • Ajout du "ET" logique
  • Ajout du "OU" logique
  • Implémentation de la détection des booléens
  • Ajout du système de gestion des opérations booléennes

@ilo80 ilo80 added the enhancement New feature or request label May 7, 2024
@ilo80 ilo80 self-assigned this May 7, 2024
@ilo80 ilo80 marked this pull request as draft May 7, 2024 11:58
@ilo80
Copy link
Contributor Author

ilo80 commented May 15, 2024

Le système de gestion des opérations booléennes permet désormais de comparer des opérations avec d'autres types.

Exemples :

  • (2 + 3) > 6 renvoie false
  • (2 + 3) > 2 + 1 renvoie true

Cependant, il n'est pas encore possible de comparer des opérations plus complexes... Ex : (2 + 3) ^ 2 > 2 + 3.

@ilo80
Copy link
Contributor Author

ilo80 commented May 16, 2024

La suppression est temporaire le temps de la ré-implémentation.

@ilo80
Copy link
Contributor Author

ilo80 commented May 16, 2024

Les opérations booléennes sont désormais implémentés dans le print !

@ilo80
Copy link
Contributor Author

ilo80 commented May 16, 2024

Il faudra ensuite les implémentés dans les variables (dans une PR à part avec le reste des opérations) + dans le futur "if"

@ilo80
Copy link
Contributor Author

ilo80 commented May 17, 2024

Je pense qu'on est good, il faudra juste à l'avenir faire des fix des opérations dans les opérations booléennes car tout ne fonctionne pas encore correctement.

@ilo80 ilo80 marked this pull request as ready for review May 17, 2024 13:35
@ilo80 ilo80 requested a review from Thotoman May 17, 2024 13:35
@ilo80 ilo80 requested a review from Itchybum-kasuga May 17, 2024 13:36
Copy link
Contributor

@Thotoman Thotoman left a comment

Choose a reason for hiding this comment

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

Ca fait beaucoup la non

@ilo80
Copy link
Contributor Author

ilo80 commented May 17, 2024

Ca fait beaucoup la non

Tkt x)

@ilo80 ilo80 merged commit 158c517 into master May 17, 2024
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants