Skip to content

MathGame - #724

Open
raonieqr wants to merge 3 commits into
the-csharp-academy:mainfrom
raonieqr:main
Open

MathGame#724
raonieqr wants to merge 3 commits into
the-csharp-academy:mainfrom
raonieqr:main

Conversation

@raonieqr

Copy link
Copy Markdown

No description provided.

@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@raonieqr Thanks for submitting! This needs an issue in the dashboard before we can approve 😁.

@TheCSharpAcademy TheCSharpAcademy self-assigned this Jul 24, 2026
@raonieqr

Copy link
Copy Markdown
Author

@raonieqr Thanks for submitting! This needs an issue in the dashboard before we can approve 😁.

Oh, right, sorry. I registered there.

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@raonieqr Thanks for submitting! It's looking good but one requirement is not being met. You're allowing non-integer divisions:

Image

Please fix so we can review 😊Let us know if you have questions.

@raonieqr

Copy link
Copy Markdown
Author

@raonieqr Thanks for submitting! It's looking good but one requirement is not being met. You're allowing non-integer divisions:

Image Please fix so we can review 😊Let us know if you have questions.

I'll make the corrections and create another pull request. Thanks for taking a look at the project.

@raonieqr raonieqr closed this Jul 26, 2026
@TheCSharpAcademy

Copy link
Copy Markdown
Collaborator

@raonieqr You don't need to create a pull request. You can modify your branch and commit and this pull request will be updated.

@raonieqr raonieqr reopened this Jul 28, 2026

@TheCSharpAcademy TheCSharpAcademy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@raonieqr Thanks for fixing divisions. I'm sorry to be a pain, but something is wrong with the scores. It gives me points even if all my answers are wrong:

Image

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