-
Notifications
You must be signed in to change notification settings - Fork 437
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
MAINT: Drop support for Python 3.4 #1866
Conversation
.travis.yml
Outdated
@@ -35,7 +34,7 @@ matrix: | |||
env: | |||
- DEPENDS="$DEPENDS scipy" | |||
# To test minimum dependencies for Python 3.6: | |||
- python: 3.4 | |||
- python: 3.6 |
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.
I think that this was a typo, now fixed.
Codecov Report
@@ Coverage Diff @@
## master #1866 +/- ##
==========================================
+ Coverage 84.31% 84.33% +0.01%
==========================================
Files 117 117
Lines 14234 14252 +18
Branches 2251 2255 +4
==========================================
+ Hits 12001 12019 +18
Misses 1716 1716
Partials 517 517
|
Looks like this one is now passing and ready for review. |
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.
I don't think we need to wait Appveyor so I will go ahead and merge it.
Thanks @arokem for this!
env: | ||
# Check these values against requirements.txt and dipy/info.py | ||
- DEPENDS="cython==0.29 numpy==1.8.2 scipy==1.0 nibabel==2.4.0 h5py==2.4.0" | ||
- DEPENDS="cython==0.29 numpy==1.9.0 scipy==1.0 nibabel==2.4.0 h5py==2.4.0" |
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.
Finally, I have one comment: why do you update this Numpy version ? If we really need to update it, can you do it for requirements.txt
and dipy/info.py
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.
Because there is no wheel for python 3.5/numpy 1.8.2.
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.
make sense, thanks!
Thank you @arokem |
Closes #1865