Skip to content

Conversation

justinGilmer
Copy link
Contributor

Previously, when breaking API changes occurred to the MoSDeF tools, a lot
of our notebooks would break silently.

Without automated testing, this can be quite difficult to keep track of,
and it is not uncommon for these notebooks to quickly fall out of sync
with the current updates.
This PR introduces a GitHub Action to create a conda environment, and
use the pytest plugin nbval to run tests on each notebook. This will
occur daily, for pushes and pull requests.

Previously, when breaking API changes occured to the MoSDeF tools, a lot
of our notebooks would break silently.

Without automated testing, this can be quite difficult to keep track of,
and it is not uncommon for these notebooks to quickly fall out of sync
with the current updates.
This PR introduces a GitHub Action to create a conda environment, and
use the pytest plugin `nbval` to run tests on each notebook. This will
occur daily, for pushes and pull requests.
Copy link

@jennyfothergill jennyfothergill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need the sanitize hook if we are running nbval-lax? (I think the lax option doesn't compare the outputs and only fails on error.)

@jennyfothergill
Copy link

Also, maybe we should pin some versions in our environments.

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