Skip to content

Added modulo operator and tests for modulo#177

Merged
WannabeMO merged 1 commit intomainfrom
mohammed
May 22, 2025
Merged

Added modulo operator and tests for modulo#177
WannabeMO merged 1 commit intomainfrom
mohammed

Conversation

@WannabeMO
Copy link
Collaborator

Finally created modulo for our interpreter. Plus the associated tests required.

Copy link
Owner

@audio-engineer audio-engineer left a comment

Choose a reason for hiding this comment

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

Just remove editor.xml from the commit, everything else is ok 👍

@WannabeMO WannabeMO force-pushed the mohammed branch 3 times, most recently from 71a124b to 014659a Compare May 19, 2025 06:22
@WannabeMO WannabeMO requested a review from audio-engineer May 19, 2025 06:24
Copy link
Owner

@audio-engineer audio-engineer left a comment

Choose a reason for hiding this comment

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

File issues

Copy link
Collaborator

@MrLoverZ MrLoverZ left a comment

Choose a reason for hiding this comment

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

lgtm!

@WannabeMO WannabeMO merged commit 157f5a2 into main May 22, 2025
1 check passed
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.

3 participants