-
Notifications
You must be signed in to change notification settings - Fork 364
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
Make sure both class and __init__ docstring are included #418
Conversation
Sure. I thought I used the correct syntax (I did not find another way to do it). If this is not correct let's fix it. |
It's something like I tried again to make the cross-linking work for function arguments and return types, but I guess I simply don't understand why the So, I don't know how to resolve this... Well, I'm assuming that we don't want to throw away all the manually organized rst files in favour of a boring tree like this: |
Actually, on second thought... In addition to # Include a separate entry for special methods, like __init__, where provided.
autodoc_default_options = {
'member-order': 'bysource',
'special-members': True,
'exclude-members': '__dict__,__weakref__'
} |
7e30c96
to
db8b849
Compare
@MainRo Yes I guess I would like all dunderscore methods, when docstrings are provided, to be included by sphinx. I've amended this PR, but since it changed after you approved it, maybe you want to reconsider or reaffirm? |
db8b849
to
f48c6c9
Compare
Thanks! |
As mentioned in #270 (comment).
By the way, I'm getting warnings about the guzzle theme, it appears they're using an outdated means of injecting script. I've created an issue.
Also, as part of the "final pass" of the docs, I think it would be good to change references to classes and methods using the appropriate markup, this will render them properly and link where possible.
And speaking about links, I will see if I can get the cross-links to work for function parameter types, today. I think that would be a great improvement, but maybe that's just me...