-
Notifications
You must be signed in to change notification settings - Fork 25
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
doc: use sphinx-autoapi for API reference #1952
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1952 +/- ##
==========================================
- Coverage 88.37% 85.96% -2.41%
==========================================
Files 89 89
Lines 10251 10251
==========================================
- Hits 9059 8812 -247
- Misses 1192 1439 +247 |
@jorgepiloto, something I need to mention (already discussed with Revathy about this too). In ansys-sphinx-theme 1.2.2 (which is the current default on this repo), the autoapi extension treats app.confdir correctly (as a path object) but using sphinx 7.1.0 will result in an error because of the above reason. In order words, we need sphinx 8.1.3 to be enforced moving forward. |
Thanks for the research, @moe-ad. Then, let's focus on updating the rules for the moment. After that, we can update the examples to Jupytext. This should allow us to revisit the complete set of examples, making sure they run without random failures. Luckly, we can migrate to the latest Sphinx after that and merge this. |
dccf82b
to
ed2b6e8
Compare
a9edf40
to
f23c959
Compare
6c2b587
to
578e582
Compare
578e582
to
c8765df
Compare
This PR is related to ansys-internal/pyansys-planning#100. The migration of the API documentation migration is being handled here.