-
Notifications
You must be signed in to change notification settings - Fork 19
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
SC80141 | Fix ReadTheDocs Build #153
SC80141 | Fix ReadTheDocs Build #153
Conversation
@@ -4,16 +4,12 @@ sphinx: | |||
configuration: doc/conf.py | |||
|
|||
python: | |||
install: | |||
- requirements: requirements-ci.txt |
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.
Is the idea that we won't be running CI on this repo anymore?
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.
@AndrewGardhouse correct me if I'm wrong but CI will still run. I think the idea is that all requirements for RTD should be encapsulated in doc/requirements.txt
so installing requirements-ci.txt
should not be needed.
Force merging this now after our detailed discussion on Slack about the necessity of these changes. Thanks @AndrewGardhouse !! |
Description of the Change:
sphinxcontrib-applehelp
,sphinxcontrib-bibtex
,sphinxcontrib-devhelp
,sphinxcontrib-htmlhelp
,sphinxcontrib-qthelp
, andsphinxcontrib-serializinghtml
versions to work with the version of sphinx used for docs.readthedocs.yml
to3.9
format
andtest
workflows if only RTD is being updatedBenefits:
RTD builds without errors, and the navbar from pennylane.ai (see https://xanaduai-pennylane--153.com.readthedocs.build/projects/strawberryfields/en/152/)
Possible Drawbacks:
N/A
Related GitHub Issues:
N/A