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

475 fix rtd build #576

Merged
merged 2 commits into from Sep 10, 2020
Merged

475 fix rtd build #576

merged 2 commits into from Sep 10, 2020

Conversation

vermeeren
Copy link
Collaborator

Closes #475. (hopefully)

Use the latest Python 3 instead of a specific minor version, install
project's requirements properly and explicitly configure the sphinx
configuration file.
@vermeeren
Copy link
Collaborator Author

Guess I'll review myself. :(

@vermeeren vermeeren self-assigned this Sep 10, 2020
@vermeeren vermeeren added bug Problem in existing code documentation Generated html, changelog, inline comments regression Something broke that worked in the past labels Sep 10, 2020
@vermeeren
Copy link
Collaborator Author

I changed the default branch in RTD settings to this branch and it built properly, even though it still installs sphinx<2 and a bunch of other crap prior to installing our requirements. That's probably hardcoded somewhere for legacy projects.

It works so I'm gonna merge this.

@vermeeren vermeeren merged commit a6c7426 into master Sep 10, 2020
@vermeeren vermeeren deleted the 475-fix-rtd-build branch September 10, 2020 20:40
vermeeren added a commit that referenced this pull request Sep 10, 2020
@michaeljones
Copy link
Collaborator

Looks good. Thank you for handling it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Problem in existing code documentation Generated html, changelog, inline comments regression Something broke that worked in the past
Projects
None yet
Development

Successfully merging this pull request may close these issues.

readthedocs.org Out of sync
2 participants