-
Notifications
You must be signed in to change notification settings - Fork 50
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
Switch to pytest runner #763
Conversation
Note: I biased this PR off of #761 since it will be merged soon and fixes some issues with the testing environment. |
…ing things as failed
a1f2f13
to
7fe576b
Compare
Awesome, works right away! Minor things:
By the way, this might be a good moment to enable tests for the |
Thanks! I will go through and fix those warnings on this PR since they really clog up the CI results. I've made a plan here to add the examples to the CI here #766 |
As a first pass in improving our testing, we are going to switch to using
pytest
to run tests. Because of the compatibility betweenpytest
andnose
we shouldn't need to do anything to the old tests, but we should start writing new tests usingpytest
features.