-
Notifications
You must be signed in to change notification settings - Fork 85
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
Make notebook tester into package #1208
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.
Nice!
…H/packagify-nb-tester
Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
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.
Thanks! Readme needs to be updated with new instructions for updating the TOML file when you have a notebook that submits jobs
raise ValueError( | ||
f"Couldn't read config from {path}; check it exists and the" | ||
" entries are correct." | ||
) from err |
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.
nice use of from err
!
Makes the notebook tester into an installable package so we can re-use it in other repos. Main changes are: * Adding `pyproject.toml` and moving script to `qiskit_docs_notebook_tester/__init__.py` * Moving the lists of notebooks to a config file * Splitting the requirements needed for running the code in the notebooks form those neeed by the testing script itself The interface via `tox` should be unaffected. --------- Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Makes the notebook tester into an installable package so we can re-use it in other repos.
Main changes are:
pyproject.toml
and moving script toqiskit_docs_notebook_tester/__init__.py
The interface via
tox
should be unaffected.