-
Notifications
You must be signed in to change notification settings - Fork 8
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
Review and refinement of API docs #177
Conversation
Working on making the API docs more polished. Closes #32.
I think I've gone about as far as I'd like to go on this. There are surely objects and methods that could use better documentation, but this is better done gradually as we notice gaps than trying to do it in one PR. |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #177 +/- ##
==========================================
+ Coverage 81.98% 82.01% +0.03%
==========================================
Files 21 21
Lines 2681 2680 -1
==========================================
Hits 2198 2198
+ Misses 483 482 -1
☔ View full report in Codecov by Sentry. |
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.
Looks good to me @dotsdl. One thing I might add is mentioning the permissive MIT licensing in developer docs, or even on home page of the docs (only if you like).
Thanks @hmacdope! Added this to the top-level index for the docs! |
Working on making the API docs more polished.
Closes #32.