-
Notifications
You must be signed in to change notification settings - Fork 23
fix(api): Make locations a publicly documented class #2375
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(api): Make locations a publicly documented class #2375
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #2375 +/- ##
==========================================
+ Coverage 84.10% 84.11% +0.01%
==========================================
Files 91 91
Lines 10529 10540 +11
==========================================
+ Hits 8855 8866 +11
Misses 1674 1674 |
The documentation for this pull request will be available at https://dpf.docs.pyansys.com/pull/2375. Please allow some time for the documentation to be deployed. |
4bc2713
to
a560270
Compare
Some tests with 'continue-on-error: true' have failed:
|
@PProfizi I approved but saw that post is broken, I guess that a PR is needed there following this one |
Yes, I will adapt Post right away, thanks! |
This PR has been closed. Documentation for this pull request will shortly be removed from its former deployment address. |
Answers part of #2009
Until now the
locations
enum has not been available in the API reference since it is defined inansys/dpf/gate/common
which is excluded from the API reference.This is very problematic as we reference locations all the time in the rest of the doc but the Enum in itself turns out to be hidden.