-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DEP: Loosely pin Cython to 3.0 #56993
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
DEP: Loosely pin Cython to 3.0 #56993
Conversation
@@ -8,7 +8,7 @@ dependencies: | |||
|
|||
# build dependencies | |||
- versioneer[toml] | |||
- cython=3.0.5 | |||
- cython~=3.0.5 |
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.
Any reason for pinning to 3.0 instead of a more general >= 3.0
range?
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 meant to put this in the OP but forgot: I brought this up at the dev meeting in December. I am wondering if minor releases should be benchmarked first before we upgrade. The general consensus was that patch versions are fine to automatically take, but there was uncertainty around minor versions.
No opposition to >=
if people are comfortable with that.
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.
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.
This means any 3.0.x? Neat.
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.
OK cool. I have no objection to this, as long as someone takes the time to do it (your benchmarks were very helpful with the 3.x upgrade)
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.
Sure let's try this out. If we find patch version are causing CI troubles often then we can reconsider
Thanks @rhshadrach |
Ah sorry should have looked at the failing check more closely, could you fix the pinning in the |
Yea - just noticed it. On it. |
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.