-
Notifications
You must be signed in to change notification settings - Fork 51
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
CI failure for python 2.7 #320
Comments
+1 for dropping Python 2.7! I usually target non-EOL Python releases (currently 3.8-3.12) in my projects. I doubt we've made any changes that would directly affect the minimum versions of other dependencies, but if we bump the minimum Python version, we might at least need to bump the minimum NumPy version. |
I asked on Twitter - in case someone has a real strong need/justification for python2.7 |
(Completely biased) Twitter sample of two (so far) suggests that dropping py2.7 is fine |
Python2.7 has been dropped from the CI - so this issue is fixed (more accurately, not relevant) |
General information
Issue description
The latest dependabot PR highlighted that CI is not running on osx-arm64 for python2.7. Given that Corrfunc supports python2.7, CI should run
Expected behavior
CI runs
Actual behavior
CI fails on
osx-arm64
. One failure is forpy2.7
, while others are forpy3.7
andnumpy1.16
What have you tried so far?
N/A
Minimal failing example
N/A
@lgarrison Maybe it's time to drop python2.7 and adopt a policy to update the minimum versions of required dependencies.
The text was updated successfully, but these errors were encountered: