Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Mark Cheong Chee Han Section: Shen Nan Crew #581

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

franktakaema888
Copy link

Please fill out the survey before submitting the pull request. Thanks!

🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀

How many hours did you spend on this assignment?
6 Hours
Please fill in one error and/or error message you received while working on this assignment.
I got some variable type error when trying to parse through the array.

What part of the assignment did you spend the most time on?
I spend the most time on the function where it allows the player to choose which dice comes first

Comfort Level (1-5):
4
Completeness Level (1-5):
4
What did you think of this deliverable?
I think it was very well done as I got to practice my control flow as well as array handling

Is there anything in this code that you feel pleased about?
My code structure? cause i'm a bit picky about tidyness

What's one aspect of your code you would like specific, elaborate feedback on?

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.

1 participant