Skip to content

Update vendored library: packaging #7559

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

Merged

Conversation

chrahunt
Copy link
Member

@chrahunt chrahunt commented Jan 6, 2020

Progresses #6908. Split from #7354.

Since the new packaging has types, it includes a py.typed. No harm in
including this in our package, and it may facilitate debug tool usage on
an installed pip by signaling that pip._vendor.packaging is
type-annotated.
@chrahunt chrahunt added project: vendored dependency Related to a vendored dependency type: maintenance Related to Development and Maintenance Processes labels Jan 6, 2020
@chrahunt chrahunt marked this pull request as ready for review January 6, 2020 19:30
@chrahunt chrahunt merged commit 902ee9b into pypa:master Jan 6, 2020
@chrahunt chrahunt deleted the maint/use-packaging-tags-update-vendored branch January 6, 2020 22:03
@@ -21,6 +21,7 @@ exclude tox.ini
exclude noxfile.py

recursive-include src/pip/_vendor *.pem
recursive-include src/pip/_vendor py.typed
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, I don't think there's much point in doing this -- we're shadowing the relevant modules through .pyi files in the vendor directory.

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Feb 6, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation project: vendored dependency Related to a vendored dependency type: maintenance Related to Development and Maintenance Processes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants