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

Fix for ReadtheDocs build #1061

Merged
merged 2 commits into from Sep 21, 2023
Merged

Fix for ReadtheDocs build #1061

merged 2 commits into from Sep 21, 2023

Conversation

ericwb
Copy link
Member

@ericwb ericwb commented Sep 20, 2023

The ReadtheDocs build is currently failing because it is unable to find sphinx_copybutton. This occurs because ReadtheDocs by default will only install dependencies of the default requirements.txt at the root.

The fix needs to install the sphinx_copybutton package found in doc/requirements.txt. ReadtheDocsv2 now provides a better configuration using a YAML file [1] instead of just the UI and its limited options.

This change adds the new .readthedocs.yaml file that allows us to control what dependencies are installed via two requirements.txt files, one in root and one in doc/.

[1] https://docs.readthedocs.io/en/stable/config-file/v2.html#build-os

The ReadtheDocs build is currently failing because it is unable
to find sphinx_copybutton. This occurs because ReadtheDocs by
default will only install dependencies of the default requirements.txt
at the root.

The fix needs to install the sphinx_copybutton package found in
doc/requirements.txt. ReadtheDocsv2 now provides a better configuration
using a YAML file [1] instead of just the UI and its limited options.

This change adds the new .readthedocs.yaml file that allows us to
control what dependencies are installed via two requirements.txt files,
one in root and one in doc/.

[1] https://docs.readthedocs.io/en/stable/config-file/v2.html#build-os

Signed-off-by: Eric Brown <eric_wade_brown@yahoo.com>
@ericwb
Copy link
Member Author

ericwb commented Sep 20, 2023

Link to the latest failed build of the docs: https://readthedocs.org/projects/bandit/builds/21971393/

@ericwb ericwb merged commit 02faada into PyCQA:main Sep 21, 2023
11 checks passed
@ericwb ericwb deleted the readthedocs branch September 21, 2023 23:00
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.

None yet

2 participants