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 Kendall #5

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

Conversation

mkendall42
Copy link

I generally felt comfortable during the entirety of the challenge. I was able to pace myself on each iteration to have a sense whether I was on track to completion within our allotted time. I focused on TDD, doing my best to (usually) first write tests and then the corresponding class methods / related code in a way that would make the tests pass. With remaining time, I refactored code, cleaned up messier segments, and strengthened the tests to minimize potential issues like false positives. I'm pleased with the outcome; it would probably be wise for me to commit slightly more often (though I started increasing the rate as I reached more involved / 'tricky' methods).

@memcmahon
Copy link
Contributor

Love that you focused on TDD for this challenge!! This one can be 'fun' to get the test stetup into a situation that will ensure no false positives 😄

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