-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
fix #2336 autosummary imported members #3235
Conversation
Can you add a test case for this change? |
Ah, OK. BTW, I think this is a "incompatible change" and this avoid imported class/function object even if it is intended. IMO, new default behavior is OK, however we should have a option for keeping previous behavior. |
OK, so I don't continue diving into the Sphinx test suite (which is very
interesting to explore, but I have also other things to do ;-)
Yes, an option would be good, it should be settable at least in the
autosummary directive, and maybe also in the conf.py
…On 13/12/16 17:32, Takayuki SHIMIZUKAWA wrote:
Ah, OK. |generate_autosummary_docs| have no test. So, now I merge this.
BTW, I think this is a "incompatible change" and this avoid imported
class/function object even if it is intended. IMO, new default behavior
is OK, however we should have a option for keeping previous behavior.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3235 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEM-J0JDzVPBvovfYdw0R7ShNJ7KxpKRks5rHrp7gaJpZM4LLgKB>.
|
|
So how can I get the functionality of |
This fixes #2336