-
Notifications
You must be signed in to change notification settings - Fork 229
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
deps: support python 3.12 #2270
Conversation
8265804
to
26ed78d
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2270 +/- ##
=======================================
Coverage 86.91% 86.91%
=======================================
Files 229 229
Lines 42150 42152 +2
Branches 7787 7787
=======================================
+ Hits 36635 36637 +2
+ Misses 4866 4865 -1
- Partials 649 650 +1 ☔ View full report in Codecov by Sentry. |
@@ -1,4 +1,5 @@ | |||
from itertools import product | |||
from . import _version |
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.
All below is autogenerated by versioneer, those files were outdated and not compatible with python3.12
- name: Set pip flags for latest python (3.12) | ||
run: | | ||
if [ "${{ matrix.python-version }}" == '3.12' ]; then | ||
echo "PIPFLAGS='--break-system-packages'" >> $GITHUB_ENV |
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.
Starting Python 3.12, anything that. is not a venv
is considered an externally managed environment and pip throws errors without that flag
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.
uncontroversial, thanks
No description provided.