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

?? shall show docstring if source info is not available #10532

Merged
merged 1 commit into from May 11, 2017

Conversation

Projects
None yet
3 participants
@blackmagic02881
Contributor

blackmagic02881 commented May 11, 2017

this is to fix the issue #10531. the reason is that when there is no source info. detail_level > 0 shall still show docstring.

@Carreau

This comment has been minimized.

Show comment
Hide comment
@Carreau

Carreau May 11, 2017

Member

Seem reasonable to me.

Member

Carreau commented May 11, 2017

Seem reasonable to me.

@takluyver takluyver merged commit 4fa9620 into ipython:master May 11, 2017

4 checks passed

codecov/patch 100% of diff hit (target 0%)
Details
codecov/project Absolute coverage decreased by -<.01% but relative coverage increased by +33.36% compared to fb1ba5e
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@takluyver takluyver added this to the 6.1 milestone May 11, 2017

@takluyver

This comment has been minimized.

Show comment
Hide comment
@takluyver

takluyver May 11, 2017

Member

And to me :-)

Member

takluyver commented May 11, 2017

And to me :-)

@Carreau

This comment has been minimized.

Show comment
Hide comment
@Carreau

Carreau May 11, 2017

Member
Member

Carreau commented May 11, 2017

@takluyver

This comment has been minimized.

Show comment
Hide comment
Member

takluyver commented May 11, 2017

Sure, #10534

@Carreau

This comment has been minimized.

Show comment
Hide comment
@Carreau

Carreau May 11, 2017

Member
Member

Carreau commented May 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment