-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Show 'home-page' project URL when Home-Page metadata value is not set #11221
Comments
Perhaps it is better to match PyPI’s display. This is how warehouse does it: ( |
Do you want to see all project URLs (without passing I like Warehouse's logic otherwise. |
There’s already discussion on that #11138. It’s probably best to show one by default, and more in verbose mode. (I wouldn’t include |
Hi, Given pep 621, this issue is going to increasingly get attention. This is my suggestion: pip show (non-verbose) should show either but not both: a. Home-page if it exists, in usual place:
at end:
b. or (if Home-page doesn't exist):
with one of Project-URLs [Homepage, Documentation, Source] on one line at end, e.g.
c. or Home-page empty, otherwise
What this does is maintain pip show for all the pre-existing packages using setup.py, setup.cfg. I don't think there is an ideal solution. This means pip show will be inconsistent, sometimes showing I'm happy to write the changes and make a pull request if you agree? |
Pull requests are always welcomed. |
What's the problem this feature will solve?
When installing a pyproject.toml-only project, the
Home-Page
metadata won't be set (as there's no way to specify it: PEP 621 only maps toProject-Url
). This meanspip show
won't provide any URL to the user.Describe the solution you'd like
If
Home-Page
is not set, instead select one of the project URLs (if present), with the following order of preference:Alternative Solutions
pip show -v
shows all project URLs, but that still shows an emptyHome-page:
fieldAdditional context
Project URLs were included in
pip show -v
in #10827Code of Conduct
The text was updated successfully, but these errors were encountered: