This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Jessica.McKellar
Recipients Jessica.McKellar, docs@python, elias, jesstess, terry.reedy
Date 2014-04-14.14:43:47
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1397486628.48.0.807014364675.issue19980@psf.upfronthosting.co.za>
In-reply-to
Content
Elias, thanks for your patch!

I think it's important to add the second part of Terry's suggestion which gives the user a specific next step to take, namely:

> Try help('help') for information on recognized strings or help(str) for help on the str class.

Can you add that to your patch?

Additionally, we'll want to make sure we don't accidentally break this new functionality. Can you add a few test cases, for example what happens when you run help on a module (e.g. help("os"), 2) help on an instance of a class (e.g. help(1)), and help on a string that doesn't have a special meaning, (e.g. help("abcxyz"))?

I don't see any existing tests for help(), but it is an instance of site._Helper (as reported by type(help)), and site tests live in Lib/test/test_site.py. It also gets loaded into builtins, so tests could also live in Lib/test/test_builtins.py.
History
Date User Action Args
2014-04-14 14:43:48Jessica.McKellarsetrecipients: + Jessica.McKellar, terry.reedy, jesstess, docs@python, elias
2014-04-14 14:43:48Jessica.McKellarsetmessageid: <1397486628.48.0.807014364675.issue19980@psf.upfronthosting.co.za>
2014-04-14 14:43:48Jessica.McKellarlinkissue19980 messages
2014-04-14 14:43:47Jessica.McKellarcreate