Skip to content
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

Remove nbsphinx from conf.py #217

Merged
merged 2 commits into from
Sep 10, 2020
Merged

Remove nbsphinx from conf.py #217

merged 2 commits into from
Sep 10, 2020

Conversation

alex-l-kong
Copy link
Contributor

What is the purpose of this PR?

The most recent readthedocs config for some reason throws a color error with the nbsphinx extension. We won't be needing Jupyter notebook support for our .md files anyways, so we'll need to delete them.

How did you implement your changes

Change the extensions variable in conf.py to not include nbsphinx and nbsphinx_link.

@alex-l-kong alex-l-kong self-assigned this Sep 10, 2020
@ngreenwald ngreenwald merged commit b142c51 into master Sep 10, 2020
@ngreenwald ngreenwald deleted the nbsphinx_delete branch September 10, 2020 20:48
alex-l-kong added a commit that referenced this pull request Jan 14, 2021
* Fix error with nbsphinx that causes a color error on readthedocs: we don't need these libraries

* Change requirement to mibilib
y2kbugger pushed a commit that referenced this pull request Jul 29, 2021
* Fix error with nbsphinx that causes a color error on readthedocs: we don't need these libraries

* Change requirement to mibilib
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