-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add testing documentation #282
Conversation
Looks promising upon first glance! |
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.
very nice.
I may have commented on things that have changed already. I tried refreshing things to avoid creating unnecessary work. |
Looks like we're ready for squashing. If you're happy with it and it renders correctly, squash and pound it. |
This covers both Python-based unittest tests and Catch2 unit tests.
Looked good to me. #'d and mergin |
Addresses #205
Addresses #222
Addresses #226
Possible contender for least glamorous PR in history :)