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

27-3 Josh #500

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

27-3 Josh #500

wants to merge 7 commits into from

Conversation

siegefried
Copy link

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

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

How many hours did you spend on this assignment?
6h

Please fill in one error and/or error message you received while working on this assignment.
On line 86, I used = instead of ==, and it turned my playerCardOne.name into true or false

What part of the assignment did you spend the most time on?
Getting the value count for ace to work out

Comfort Level (1-5):
4

Completeness Level (1-5):
2

What did you think of this deliverable?
I wish I had more time to spruce it up, not really a good time to start a project 😢

Is there anything in this code that you feel pleased about?
N/A

What's one aspect of your code you would like specific, elaborate feedback on?
Across the 3 projects, I feel like the thing I need to improve most on is preparing for project requirements to scale upwards. Re-writing / adapting code feels like a lot of additional effort especially when it introduces bugs. Any advice on how best to prepare for such would be appreciated.

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