-
Notifications
You must be signed in to change notification settings - Fork 6
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 to support python 3.12 #172
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #172 +/- ##
=======================================
Coverage 86.04% 86.04%
=======================================
Files 5 5
Lines 1161 1161
Branches 296 288 -8
=======================================
Hits 999 999
Misses 107 107
Partials 55 55 ☔ View full report in Codecov by Sentry. |
Note: After updating the .toml, tox, and workflows, I did a search for "3.11". I'll do another look to see what else needs to be bumped to 3.12 |
@oruebel The bump to 3.12 seems to have bequeathed some gallery errors that need to be resolved. I'd like to make it a separate ticket/PR to not hold this up. Thoughts? I need to do another pass through before requesting a review. Update: I just ended up fixing it here. A new warning filter for test_gallery.py |
Review Note: I did not update the changelog since this is not a user facing update. |
Please add a changelog -- The changelog is for any change to the software, and is for both users and developers, especially to check for changes that affect compatibility (here, for example, we announce that PyNWB is officially compatible with Python 3.12 - some downstream packages do not support Python 3.12 because pynwb does not so it's important for us to announce this). In contrast, when making changes to the schema repo, the schema release notes should not be updated when there are no changes to the schema files. We could add a changelog there for all changes to the repo, but its not really necessary because the main purpose of the schema repo is to hold the schema files. The surrounding code is for internal use. |
I mean HDMF-Zarr, not PyNWB, though my example about downstream packages not supporting a version of Python because PyNWB does not officially support is is true for PyNWB, not HDMF-Zarr (AFAIK) |
Got it. Done. |
Motivation
What was the reasoning behind this change? Please explain the changes briefly.
Support python 3.12
How to test the behavior?
Checklist
ruff
from the source directory.