-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Tutorial for extending autodoc #8972
Conversation
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.
LGTM with nits.
.. autointenum:: my_enums.Colors | ||
|
||
|
||
.. todo:: Document available Documenter classes, options... |
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.
I think this is not a topic of tutorials. It would be better to separate class references and tutorials.
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.
Do you want me to remove this todo?
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.
After going through the autodoc source code I got a very strong urge to make changes to the code. I want to ask the mailing list first about my proposed changes. I will probably create a separated API page after making changes.
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.
Do you want me to remove this todo?
Yes. It's better to document it on another page.
@tk0miya Do you want me to push new commits or amend? |
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.
LGTM with nits.
Uses :ref: link because :doc: does not work.
Used in add_autodocumenter docstring
Thank you for update. Looks good. Can I merge this? |
Yes |
Merged! Thank you for your work! |
Subject: Tutorial for extending autodoc
Feature or Bugfix
Purpose
Detail
Relates