-
Notifications
You must be signed in to change notification settings - Fork 0
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: build was failing due to deprecated configs & deps compatibility issues #72
Conversation
6acd0a5
to
59749aa
Compare
96e8d37
to
59749aa
Compare
485e635
to
8c82d14
Compare
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.
I'm not so happy with adding the constraints, but I didn't find a better way to make it work. I think we can go with this now and in the future, try to simplify the documentation or make it similar to the openedx but with our logos to avoid the many constrictions.
Can you help me remove the release variable from the conf.py
? It is out of scope, but I think it is a quick fix we can add. If not, don't worry :)
a19f950
Description
This PR intends to fix the issue #70 by adding the
build.os
config key to the.readthedocs
config file as shown in readthedocs/readthedocs.org#10290 (comment). Also, this PR adds some constraints (at doc.in) so the dependencies are compatible withSphinx>5.0
Ref. https://blog.readthedocs.com/use-build-os-config/
Aditional Information
JIRA ISSUE DS-857