[go: up one dir, main page]

Skip to content
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 tests on latest Python 3.13 (and 3.12) #17849

Merged
merged 1 commit into from
Sep 28, 2024

Conversation

hauntsaninja
Copy link
Collaborator

Related to python/cpython@c6c3d97 (python/cpython#121329), thanks to brianschubert for noticing

Related to python/cpython@c6c3d97
(python/cpython#121329), thanks to brianschubert for noticing
@hauntsaninja hauntsaninja changed the title Fix tests on latest Python 3.13 Fix tests on latest Python 3.13 (and 3.12) Sep 28, 2024
@hauntsaninja
Copy link
Collaborator Author
hauntsaninja commented Sep 28, 2024

cc @picnixz in case you find instances of hyrum's law amusing :-)
(this is totally fine, cpython doesn't promise much in regards to error text)

@picnixz
Copy link
Contributor
picnixz commented Sep 28, 2024

So this is the first occurrence of me breaking something important :D 🎉 (well, now the message is a bit more correct concerning what happens behind the scene).

@hauntsaninja hauntsaninja merged commit 1a2c8e2 into python:master Sep 28, 2024
14 checks passed
@hauntsaninja hauntsaninja deleted the odict-fix branch September 28, 2024 21:20
@JelleZijlstra
Copy link
Member

I do think this is a data point that we should avoid changing error messages in bugfix releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants