Skip to content

fix: prepare for 3.14 beta 2 #5697

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

Merged
merged 1 commit into from
May 27, 2025
Merged

Conversation

henryiii
Copy link
Collaborator

@henryiii henryiii commented May 26, 2025

Description

python/cpython#133912 isn't blocking the 3.14b2 release (already tagged), so we have to extend our test skip.

The fix is in python/cpython#134725 (haven't tested it yet, but the added test looks correct).

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@@ -306,8 +306,9 @@ def test_property_rvalue_policy():
# https://foss.heptapod.net/pypy/pypy/-/issues/2447
@pytest.mark.xfail("env.PYPY")
@pytest.mark.xfail(
sys.version_info == (3, 14, 0, "beta", 1),
reason="3.14.0b1 bug: https://github.com/python/cpython/issues/133912",
sys.version_info == (3, 14, 0, "beta", 1)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sys.version_info in ((3, 14, 0, "beta", 1), (3, 14, 0, "beta", 2))

?
(also in tests/test_pickling.py)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could even do

sys.version_info[:4] == (3, 14, 0, "beta") and sys.version_info[4] in {1, 2}

But since this is a limited one-off (and we can remove it in a while, we don't support pre-releases forever), I wasn't too worried here, whatever reads best.

@henryiii henryiii merged commit 1c10d5e into pybind:master May 27, 2025
74 checks passed
@henryiii henryiii deleted the henryiii/fix/314b2 branch May 27, 2025 04:05
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label May 27, 2025
@henryiii henryiii removed the needs changelog Possibly needs a changelog entry label May 27, 2025
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.

2 participants