-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix botocore/boto3 dependency version mismatch #1801
Conversation
Coverage increased (+0.2%) to 92.413% when pulling aa5e4a5d2f7f3b008a06bdbb2c0bb5074d0700bf on meneal:fix/mismatch into e8c65d3 on spulec:master. |
setup.py
Outdated
@@ -8,7 +8,7 @@ | |||
install_requires = [ | |||
"Jinja2>=2.7.3", | |||
"boto>=2.36.0", | |||
"boto3>=1.6.16", | |||
"boto3==1.7.84", |
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.
it will force the tested app installs [email protected]
which may be unexpected.
What about "boto3>=1.6.16,<1.8",
?
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.
What about just <1.8
?
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.
Pinning boto3 to this version would remove the mismatch between boto3 and the botocore dependency in setup.py. Closes getmoto#1800
@tomelliff comments addressed. |
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.
👍 the PR subject should be updated as well
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.
Looks good. Thanks for doing this.
🔥 🔥 🔥 |
please make this merge.... I'm working on aws and I can't just use a previous version of boto. This breaks my tests to the point where I have to manually mock the resources myself, which defeats the purpose of using this lib. Hopefully you can get this pulled in asap so we can get over this problem. |
Joining @joshua-alday , this is needed ASAP. |
Pinning boto3 to this version would remove the mismatch between
boto3 and the botocore dependency in setup.py.
Closes #1800