-
Notifications
You must be signed in to change notification settings - Fork 2.2k
fix: support Python 3.14 #5646
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
fix: support Python 3.14 #5646
Changes from 19 commits
77f03f4
3bd75bd
236b32f
ce693fd
cd5125c
2cd17ba
8e4575a
5e0324c
76408fd
c5699a8
8031427
dce1fcb
e7a623e
1f2e5d6
6caea71
6e9c304
c4226a0
151f225
7bca953
87e7a50
7a4e2db
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,7 @@ def test_independent_subinterpreters(): | |
|
||
sys.path.append(".") | ||
|
||
if sys.version_info >= (3, 14): | ||
if sys.version_info >= (3, 15): | ||
import interpreters | ||
elif sys.version_info >= (3, 13): | ||
import _interpreters as interpreters | ||
|
@@ -86,7 +86,7 @@ def test_dependent_subinterpreters(): | |
|
||
sys.path.append(".") | ||
|
||
if sys.version_info >= (3, 14): | ||
if sys.version_info >= (3, 15): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this final for 3.14, or could this still change before the 3.14.0 release? If the latter: maybe add a comment? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It will continue to work. I think |
||
import interpreters | ||
elif sys.version_info >= (3, 13): | ||
import _interpreters as interpreters | ||
|
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 like the suppressed due to low confidence comment, but instead we should just use the HEX version like everywhere else.