-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
Move old C API tests for floats to Lib/test/test_capi/ #111765
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
Labels
Comments
skirpichev
added a commit
to skirpichev/cpython
that referenced
this issue
Nov 6, 2023
…float.py Move test_pack/unpack()
serhiy-storchaka
pushed a commit
that referenced
this issue
Nov 7, 2023
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this issue
Nov 7, 2023
…float.py (pythonGH-111766) (cherry picked from commit a077b2f) Co-authored-by: Sergey B Kirpichev <[email protected]>
serhiy-storchaka
pushed a commit
that referenced
this issue
Nov 7, 2023
…_float.py (GH-111766) (GH-111818) (cherry picked from commit a077b2f) Co-authored-by: Sergey B Kirpichev <[email protected]>
hugovk
pushed a commit
to hugovk/cpython
that referenced
this issue
Nov 8, 2023
@serhiy-storchaka, I think this could be closed. The test_float.py still imports cpython/Lib/test/test_float.py Lines 663 to 668 in a077b2f
but this is rather a bug in sys.float_info. |
aisk
pushed a commit
to aisk/cpython
that referenced
this issue
Feb 11, 2024
Glyphack
pushed a commit
to Glyphack/cpython
that referenced
this issue
Sep 2, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
After #111624, we have C API tests for PyFloat_* both in Lib/test/test_float.py (pack/unpack tests) and Lib/test/test_capi/test_float.py. I think we should move remaining tests to Lib/test/test_capi/.
Linked PRs
The text was updated successfully, but these errors were encountered: