-
Notifications
You must be signed in to change notification settings - Fork 4.1k
feanil/fix docs build #36259
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
feanil/fix docs build #36259
Conversation
feanil
commented
Feb 14, 2025
- build: Add more apps to the set of installed apps.
- build: We don't need to override this setting.
- docs: Update the Open API docs.
These app models get pulled in when trying to build the OpenAPI docs so we need their relevant apps installed into the environment used to generate the docs.
docs/conf.py
Outdated
@@ -112,7 +112,7 @@ | |||
# You can specify multiple suffix as a list of string: | |||
# | |||
# source_suffix = ['.rst', '.md'] | |||
source_suffix = '.rst' | |||
# source_suffix = '.rst' |
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.
Should this just get deleted?
Are the Open API docs generated somehow or are these manual edits? |
The format of the `source_suffix` setting has change but the default is fine for us so we don't need to override this.
f6572bd
to
b079d4c
Compare
The API docs are generated as a part of running the |
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 sure how relevant the quality errors are, but this otherwise seems good
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |