-
Notifications
You must be signed in to change notification settings - Fork 421
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
Merge API documentation in Sphinx with docstrings #126
Conversation
For your reviewing benefit, a PDF render. |
I have no idea how hard this rebase is going to be, but there's a merge conflict :( |
oh yeah, this looks interesting. please rebase. 🐶 |
could you also fix
in |
:py:class:`CRL` :py:class:`CRLType` | ||
========================= ============================= | ||
|
||
Some objects, such as py:class`Revoked`, don't have ``Type`` |
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.
broken markup (missing : before py)
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.
Fixed in 5faa281
That’s all I could find. Maybe after you’ve fixed those, @reaperhulk or @alex can eyeball the docs too if I’ve missed something and then we can merge. :) |
Thanks for the review, @hynek! I've addressed all comments. |
2 similar comments
2 similar comments
4 similar comments
hm, why did coverage decrease by 2.35%? :( |
Bump, I would really like to merge this so that new API entries in other PRs can use this. Any idea why the coverage decreased? :-/ |
Merge API documentation in Sphinx with docstrings
Major improvements to the docstrings (mostly by merging the information that was only in Sphinx, but also grammar fixes).
This isn't entirely complete: sign, verify, PKCS7... still have their own API docs in Sphinx. But hey, this PR is already more than big enough (thanks @alex for nudging me to send it in quicker).
This fixes #125, too.