-
Notifications
You must be signed in to change notification settings - Fork 82
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
added FinishWithTentsTest #676
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good. Project builds just fine with the new unit tests. Comments are clear but a visual representation of the puzzle may be helpful (i.e. draw the grid and mark TENT
or GRASS
tiles).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without a working dev I can;t really comment on this. But I see it has one positive review, so I'll go ahead and approve :P
Alright, I'll merge this PR then. |
Description
added tests to FinishWithTentsTest
added comments to FinishWithTentsTest
Closes #604
Type of change
How Has This Been Tested?
Checklist: