-
Notifications
You must be signed in to change notification settings - Fork 169
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
[SCSB-170] pin Python<3.13 #8703
Conversation
2ec484d
to
c5aa127
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8703 +/- ##
==========================================
+ Coverage 60.35% 60.38% +0.02%
==========================================
Files 372 372
Lines 38360 38337 -23
==========================================
- Hits 23151 23148 -3
+ Misses 15209 15189 -20 ☔ View full report in Codecov by Sentry. |
Would this allow removing the additions in #8691? If so, I'd say we remove those cibuildwheel changes as the upper pin in this PR is easier to maintain. |
I assumed so when making this PR, but it |
f247592
to
958240c
Compare
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.
Thanks! I'll close #8692 as well.
Resolves SCSB-170
Checklist for PR authors (skip items if you don't have permissions or they are not applicable)
CHANGES.rst
within the relevant release sectionHow to run regression tests on a PR