Skip to content

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Jan 22, 2022

This reverts commit 57316c5 for files pertaining to importlib.metadata and importlib.resources (rationale).

https://bugs.python.org/issue46425

…t_importlib` (GH-30682)"

This reverts commit 57316c5 for files pertaining to importlib.metadata and importlib.resources.
@jaraco
Copy link
Member Author

jaraco commented Jan 22, 2022

I confirmed by syncing this change into the backports that the backports no longer show a diff w.r.t the previous change.

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks! The only difference between yours and #30800 is that I've left a comment on the very end of each file that is not suitable to be executed directly.

Sorry for a small breakage 🙂
TIL how backporting to https://github.com/python/importlib_metadata process looks like.

@jaraco
Copy link
Member Author

jaraco commented Jan 22, 2022

In #30800, we decided to proceed with option (b), so merging this now (once tests pass). Thanks!

@sobolevn
Copy link
Member

@jaraco tests most likely won't pass 🙂
See #30801

@jaraco jaraco closed this Jan 23, 2022
@jaraco jaraco reopened this Jan 23, 2022
@jaraco jaraco merged commit d888ff5 into main Jan 23, 2022
@jaraco jaraco deleted the bpo-46425/restore-simple-importlib-tests branch January 23, 2022 02:38
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.

4 participants