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

Add autogenerated api docs. #641

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

DanLipsitt
Copy link
Contributor

@DanLipsitt DanLipsitt commented Aug 21, 2017

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:

  • Prose docs about how to use OpenHTF.
  • Tying existing markdown docs into this system (I think this can be done with Sphinx).
  • Configuring ReadTheDocs.
  • Fixing syntax errors in docstrings.

This change is Reviewable

```
sphinx-quickstart --sep --project=OpenHTF --author="OpenHTF Contributors" \
                  --language=en --ext-viewcode --ext-autodoc --makefile --batchfile --suffix=.rst \
                  --master=index docs
```
@DanLipsitt
Copy link
Contributor Author

DanLipsitt commented Aug 21, 2017

This is currently interfering with setup.py build_proto. Not clear why yet. PBR is probably overriding some necessary fields from setup.py.

Copy link
Contributor

@jettisonjoe jettisonjoe left a 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?

@fahhem
Copy link
Collaborator

fahhem commented Nov 30, 2017 via email

@jettisonjoe
Copy link
Contributor

@fahhem, Maddy just added us as maintainers to the OpenHTF readthedocs project.

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.

3 participants