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

Remove the dependency on requests #112

Merged
merged 1 commit into from
Nov 8, 2018
Merged

Remove the dependency on requests #112

merged 1 commit into from
Nov 8, 2018

Conversation

Dunedan
Copy link
Contributor

@Dunedan Dunedan commented Nov 7, 2018

This removes the use of requests from the core functionality, which
results in requests being only used as extension, therefore not being
necessary as dependency anymore.

The motivation for this change is to get rid of all the dependencies of
requests, whose size sums up quite a bit, when not requiring to patch
requests. It follows the removal of requests from botocore, which
was done as part of v1.11.0.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

This removes the use of `requests` from the core functionality, which
results in `requests` being only used as extension, therefore not being
necessary as dependency anymore.

The motivation for this change is to get rid of all the dependencies of
`requests`, whose size sums up quite a bit, when not requiring to patch
`requests`. It follows the removal of `requests` from `botocore`, which
was done as part of v1.11.0.
@haotianw465 haotianw465 merged commit 3b2df2a into aws:master Nov 8, 2018
@haotianw465
Copy link
Contributor

Thank you for your contribution. I have merged this PR and it will be out with our next release.

@Dunedan Dunedan deleted the remove-requests-dependency branch January 30, 2019 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants