Skip to content

gh-138094: Fix misleading suggestions in interactive help prompt #138095

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

adqm
Copy link
Contributor

@adqm adqm commented Aug 23, 2025

This PR represents an attempt to address both of the issues raised in #138094 by changing how help(help) is handled, and by removing suggested calls to help(...) in the output when we can't find a given topic in interactive mode.

Happy to adjust if need be (or to separate this into 2 PR's if that would be better).

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's worth adding an entry to the "What's New in Python 3.15" page too, because this seems like a feature, not a bugfix.

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@adqm
Copy link
Contributor Author

adqm commented Aug 24, 2025

I think it's worth adding an entry to the "What's New in Python 3.15" page

I'm not sure, just because this PR alone represents pretty minor tweaks to help's behavior. But maybe in conjunction with #137966 if these both get merged in, it would make sense? I also have a few other ideas in mind so there may be additional help-related PR's coming when I have a chance.

Of course, happy to defer to y'all's judgment, though.

@ZeroIntensity
Copy link
Member

Alternatively, we could add a .. versionchanged note to help, because this is a behavior change. #137966 should definitely get a whatsnew entry, though.

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

Successfully merging this pull request may close these issues.

2 participants