-
Notifications
You must be signed in to change notification settings - Fork 7
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
Run Examples in CI. #578
Run Examples in CI. #578
Conversation
Fixed typo.
Using doctest, sphinx-doctest, and papermill to run these tests. |
TODO: figure out what to do with |
|
I'm thinking we should override the changelog test. |
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.
Changelog test ok.
This is a testing and documentation update. We should make the following changes:
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.
Let's add surface coefficients and then merge.
Description
This implemented testing of (almost) all example code blocks. This includes:
README.md
doc/source/*.rst
demo/*.ipynb
This also fixed all bugs found. The only code that wasn't tested is ones where the output are multiline statements. This is too complicated to work with
sphinx-doctest
.Fixes #575, fixes #571
Checklist