ni / nimi-python

Python bindings for NI Modular Instrument drivers.
Other
112 stars 84 forks source link

Install read_the_docs theme in .readthedocs.yaml #2029

Closed ni-jfitzger closed 10 months ago

ni-jfitzger commented 10 months ago

What does this Pull Request accomplish?

Documentation is failing to build due to the theme not being found. Most likely, Read the Docs changed something so that their theme is no longer installed by default.

This change updates the .readthedocs.yamls to install the theme.

I don't have time to properly add a requirements file, right now, so this is just a band-aid: a sort-of requirements file to get the dependency installed. We still have an open issue to properly add a requirements file for the sphinx build.

List issues fixed by this Pull Request below, if any.

What testing has been done?

None. If the PR checks pass, it should fix the issue.

ni-jfitzger commented 10 months ago

I clearly misinterpreted the Read the Docs documentation. I have to add a requirements file.

ni-jfitzger commented 10 months ago

Skipping the rest of the build, since the documentation builds passed and nothing else changed.