-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Remove imports from pip/requests's vendored packages #13171
Remove imports from pip/requests's vendored packages #13171
Conversation
Requests already depends on charset-normalizer so this is a "cheaper" new dependency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
charset-normalizer is a pretty old version on defaults: https://anaconda.org/main/charset-normalizer
But a recent one on conda-forge: https://anaconda.org/conda-forge/charset-normalizer
Might be useful to review https://github.com/Ousret/charset_normalizer/blob/master/CHANGELOG.md if there are any features or fixes needed.
Otherwise LGTM
Description
Stop relying on vendored packages provided by
pip
andrequests
and instead depend on the dependency itself.Removes the following fallback imports:
pip._vendor.requests
requests.packages.chardet
(addedcharset-normalizer
as dependency)pip._vendor.requests.packages.chardet
(addedcharset-normalizer
as dependency)Checklist - did you ...
Add a file to thenews
directory (using the template) for the next release's release notes?Add / update necessary tests?Add / update outdated documentation?