-
Notifications
You must be signed in to change notification settings - Fork 79
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
Update setup.py #235
Update setup.py #235
Conversation
setup.py
Outdated
'docker-py', | ||
'pip', | ||
'PyYAML', | ||
'retrying' |
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.
missing comma ,
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.
oh, thanks!
dd8c3d5
to
8d3f961
Compare
Codecov Report
@@ Coverage Diff @@
## master #235 +/- ##
=======================================
Coverage 89.06% 89.06%
=======================================
Files 30 30
Lines 1728 1728
=======================================
Hits 1539 1539
Misses 189 189 Continue to review full report at Codecov.
|
@chekunkov , it seems six>=1.7.0 is needed too. See #239 |
8d3f961
to
00b58b0
Compare
@redapple added |
Pin scrapinghub (>=1.9.0) and six (>=1.7.0) versions. Add Python 3.6 trove classifier.
00b58b0
to
065938c
Compare
Thanks guys |
Pin scrapinghub (>=1.9.0) and six (>=1.7.0) versions.
Add Python 3.6 trove classifier.
It may be a good idea to have versions for other requirements pinned to some meaningful ranges, e.g. I'm not sure shub works well with all versions of docker-py or all versions of click.
Fixes #239