Skip to content
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

Migrate 'pip list' to use metadata abstraction #9825

Merged
merged 3 commits into from
Jul 12, 2021

Conversation

uranusjr
Copy link
Member

Continuing #7413, one module at a time…

@uranusjr uranusjr added the skip news Does not need a NEWS file entry (eg: trivial changes) label Apr 24, 2021
@uranusjr uranusjr force-pushed the metadata-list-command branch 2 times, most recently from 3960026 to 5f34755 Compare April 26, 2021 06:10
This help function is much better than the previous ad-hoc logic used in
test_uninstall.py, which has trouble identifying normalized names.
@uranusjr
Copy link
Member Author

Anyone wants to take a look at this? I’m coming back to work on this importlib.metadata stuff and need to get this in before moving on.

The change has one behavioural implication: Since the new metadata backend normalises project names by default, pip list will now always output normalised names. It’s certainly possible to output the “raw” name, but I’m trying to keep the implementation simple unless someone complains about this.

Does this change deserve a changelog, and in what category?

@uranusjr uranusjr merged commit e6c3177 into pypa:main Jul 12, 2021
@uranusjr uranusjr deleted the metadata-list-command branch July 12, 2021 03:44
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
skip news Does not need a NEWS file entry (eg: trivial changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant