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

FABIAN KHOO PTBC-13-1 MATCH GAME #66

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Erilcartul
Copy link

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

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

How many hours did you spend on this assignment?- 8 hours

Please fill in one error and/or error message you received while working on this assignment. One common error I got was variables not being defined. Had to readjust code to set certain variable as global variables in order to call those variables somewhere down the code.

What part of the assignment did you spend the most time on? - spent most of my time creating a working timer. resolving some bugs with the reset game function as well as resetting the timer.

Comfort Level (1-5): 3

Completeness Level (1-5): 4 (did not include score keeping, else the rest of the features are in)

What did you think of this deliverable? pretty challenging.

Is there anything in this code that you feel pleased about?- very pleased to get the timer working despite great difficulty although I recognize that I could create a function for the timer instead of hard coding it.

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