Skip to content
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

169 improve apidoc generation #342

Merged
merged 15 commits into from
Oct 23, 2023
Merged

Conversation

surgura
Copy link
Contributor

@surgura surgura commented Oct 23, 2023

  • Remove autodoc and apidoc in favor of autoapi.
  • No need to manually edit api documentation ever again.
  • Display not perfect but reasonable for now. See Autoapi issues #341

closes #169

@surgura surgura marked this pull request as draft October 23, 2023 10:56
@surgura surgura force-pushed the 169-improve-apidoc-generation branch from 837a860 to d03d13d Compare October 23, 2023 10:58
@surgura surgura marked this pull request as ready for review October 23, 2023 11:17
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fyi api reference is programmatically added by autoapi

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file was missing but we didn't see it until now

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed the generic parameters so they don't class with the Individual class in the docs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very similar to the default used by autoapi, just added that module names don't include their complete path package1.package2.module adn just module. it's implicit anyway wrt to where you are in the docs

Copy link
Member

@oliverweissl oliverweissl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the failed CI is wanted because the system changed

@surgura
Copy link
Contributor Author

surgura commented Oct 23, 2023

I assume the failed CI is wanted because the system changed

not intentional, i'll fix it before merging

@surgura surgura merged commit 161cdf1 into development Oct 23, 2023
@surgura surgura deleted the 169-improve-apidoc-generation branch October 23, 2023 13:51
surgura added a commit that referenced this pull request Oct 31, 2023
* Remove autodoc and apidoc in favor of autoapi.
* No need to manually edit api documentation ever again.
* Display not perfect but reasonable for now. See #341
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants