-
Notifications
You must be signed in to change notification settings - Fork 14
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
Demonstrate automatic Sphinx openapi #648
base: master
Are you sure you want to change the base?
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #648 +/- ##
=======================================
Coverage 84.61% 84.61%
=======================================
Files 75 75
Lines 3750 3750
=======================================
Hits 3173 3173
Misses 577 577 ☔ View full report in Codecov by Sentry. |
I suspect this is a bug in the sphinx plugin:
|
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 think you need to also change the .readthedocs.yaml
file to make it install the added optional dependencies. I think you can use the extra_requirements
option in https://docs.readthedocs.io/en/stable/config-file/v2.html#packages
Otherwise this seems to work. Even though the look of it could be improved.
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.
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.
Thats a functioning POC 👍
There are improvements that could be made in the future:
- those mentioned above
- contrast errors should be fixed. The WAVE accessibility tool shows whooping 500 contrast errors, but it seems that they all concern the yellow and the green text as shown below:
- indentations should be fixed so that alignment of the same level elements matches, and so that indentations are more uniform in general:
- Some indentations could probably be increased for better readability in my opinion (Parameters, Status codes etc elements)
- Some indentations could be reduced for better readability in my opinion:
I don't know how much of this is controlled by our sphinx theme or whether we'd have to fork the plugin to make the changes. |
Replacing the default sphinx theme (alabaster) with "sphinx_rtd_theme" yields this: What do you think, @podliashanyk? |
Replacing it with "sphinx_rtd_theme" is also nice, since we use the same for NAV! |
The API looks like barf with sphinx_rtd_theme, the indents don't make sense. I'll add some more screenshots. |
Theme: "sphinx_rtd_theme" |
Theme: "sphinx_book_theme" This also has a darkmode-theme. |
Theme: "furo" It is possible to tune a lot with furo: https://pradyunsg.me/furo/customisation/ |
I don't think this will be ready for next release. |
Quality Gate passedIssues Measures |
For #628
Enter docs/, run "make html", look at the result in
docs/_build/html/index.html