-
-
Notifications
You must be signed in to change notification settings - Fork 948
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: add 3.13 to test matrix #3427
Conversation
Do you think it is worth it? If I remember correctly we always tested new versions only once the migration of conda-forge was sufficiently far. |
I'm inclined to just wait now - I thought I'd see if it was easy to do- and by the looks it's not super simple. We don't have any compiled code so at worst I think some deprecation warnings would surface, which we can fix after 3.13 is out. |
looks like we're waiting on pyarrow 18 to come out (we could temporarily drop pyarrow stuff from the build, but it's probably easier to just wait) |
PyArrow 18 is in the RC phase, so hopefully available something like next week. But temporarily commenting it out from the env file or installing from nightly sounds fine as well. |
I would prefer not to use nightly wheels here. It may stuck there for a while. But I am fine testing without pyarrow. |
I think I'll just leave this until pyarrow is out then - the main thing is that 3.13 is not breaking and we can see that from the last commit |
This reverts commit 33e5864.
pyarrow 18 came out yesterday, and this is now installing it and passing. |
No description provided.