Skip to content

Conversation

@serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Jan 20, 2025

They provide better error report.

Copy link
Contributor

@ncoghlan ncoghlan left a comment

Choose a reason for hiding this comment

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

Looks like a straightforward test improvement to me.

@ncoghlan
Copy link
Contributor

I was going to suggest backporting, but these methods are new in 3.14, so that's not an option.

@serhiy-storchaka serhiy-storchaka merged commit f7cc7d2 into python:main Jan 21, 2025
47 checks passed
@serhiy-storchaka serhiy-storchaka deleted the test_importlib-asserts branch January 21, 2025 09:24
@serhiy-storchaka serhiy-storchaka added the needs backport to 3.13 bugs and security fixes label Jan 21, 2025
@miss-islington-app
Copy link

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @serhiy-storchaka, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker f7cc7d296c2cbb33d3f0bde4ace82e8569f7dbc3 3.13

serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Jan 21, 2025
…est_importlib (pythonGH-129052)

(cherry picked from commit f7cc7d2)

Co-authored-by: Serhiy Storchaka <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jan 21, 2025

GH-129123 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jan 21, 2025
@serhiy-storchaka
Copy link
Member Author

I was going to suggest backporting, but these methods are new in 3.14, so that's not an option.

It is an option. See #129123.

ambv pushed a commit that referenced this pull request Apr 8, 2025
…portlib (GH-129052) (#129123)

(cherry picked from commit f7cc7d2)

Co-authored-by: Serhiy Storchaka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants