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.

Sign up for GitHub

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

Jump to bottom

Raise minimum required s3transfer version #3003

Merged
merged 1 commit into from
Nov 29, 2017
Merged

Raise minimum required s3transfer version #3003

merged 1 commit into from
Nov 29, 2017

Conversation

teeberg
Copy link
Contributor

@teeberg teeberg commented Nov 29, 2017

#2997 depended on boto/s3transfer#101 according to its description, but that PR was only released in 0.1.12:
boto/s3transfer@31b344b
image

I'm raising the minimum required version for s3transfer to make sure noone else runs into this issue when upgrading aws-cli doesn't automatically pull in the right s3transfer version:

[localhost] local: aws s3 cp [...] s3://[...]

__init__() got an unexpected keyword argument 'max_bandwidth'

@stealthycoin stealthycoin merged commit acd15b1 into aws:develop Nov 29, 2017
@teeberg teeberg deleted the raise-minimum-s3transfer-version branch November 29, 2017 23:39
@joguSD joguSD mentioned this pull request Nov 30, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kyleknap kyleknap kyleknap approved these changes

@jamesls jamesls jamesls approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@teeberg @jamesls @kyleknap @stealthycoin