-
Notifications
You must be signed in to change notification settings - Fork 86
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: Add module API documentation. See #267 #279
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #279 +/- ##
=======================================
Coverage 52.38% 52.38%
=======================================
Files 39 39
Lines 4169 4169
Branches 704 704
=======================================
Hits 2184 2184
Misses 1859 1859
Partials 126 126
Continue to review full report at Codecov.
|
@jcfr @nicain @dorukozturk Ok, I found the problem. This issue is that in 39c9a71 @dorukozturk added a |
@jcfr @nicain @dorukozturk @ajtritt I just added #280 which should fix this problem. |
Closed because #280 has resolved the original concern. |
Motivation
API documentation missing from readthedocs. See #267
Look like an alternative would be to write our builder on readthedocs, not sure if it is worth the effort. I think it is reasonable to expect developer to locally generated the documentation and checked in the update.
See https://docs.readthedocs.io/en/latest/builds.html#understanding-what-s-going-on