-
Notifications
You must be signed in to change notification settings - Fork 216
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
Add autogenerated api docs. #641
base: master
Are you sure you want to change the base?
Conversation
``` sphinx-quickstart --sep --project=OpenHTF --author="OpenHTF Contributors" \ --language=en --ext-viewcode --ext-autodoc --makefile --batchfile --suffix=.rst \ --master=index docs ```
This is currently interfering with |
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.
Can we get a hosted version of the built docs up somewhere so we can take a look?
This should be compatible with readthedocs, but the openhtf project is
owned by maddychan so I can't point it at this branch temporarily to see it
working.
…On Thu, Nov 30, 2017 at 2:46 PM, Joe Ethier ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Can we get a hosted version of the built docs up somewhere so we can take
a look?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#641 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AASrtBYCh5hErLH6RY-QrU8rEzOBm1xOks5s7zBXgaJpZM4O8yAa>
.
|
@fahhem, Maddy just added us as maintainers to the OpenHTF readthedocs project. |
This is a first crack at generating api docs from docstrings using Sphinx. I've decided to use PBR for this. I've seen it used in various other projects recently, and it seems to work well. The alternative is to use
sphinx-autogen
, but it isn't as automatic and the output is a bit messy. There are other useful things it can do as well, but I haven't configured that.Things that this PR doesn't address:
This change is