Skip to content

[3.13] gh-116608: Revert "[3.13] gh-116608: Apply style and compatibility changes from importlib_resources. (GH-123028) (#123051)" #123072

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

Closed
wants to merge 1 commit into from

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Aug 16, 2024

This reverts commit 5ac14ee.

This commit should be re-applied after 3.13.0 final.

…nd compatibility changes from importlib_resources. (pythonGH-123028) (python#123051)"

This reverts commit 5ac14ee.

This commit should be re-applied after 3.13.0 final.
@jaraco
Copy link
Member Author

jaraco commented Aug 16, 2024

I guess technically this change should also be reviewed.

@jaraco jaraco requested review from encukou and removed request for warsaw and FFY00 August 16, 2024 16:12
@FFY00
Copy link
Member

FFY00 commented Aug 16, 2024

@jaraco feel free to request a review from me in such cases. I do tend to keep up with the ongoing work in the importlib_metadata and importlib_resources backports, so it might be easier for me to have a look compared to others.

@jaraco
Copy link
Member Author

jaraco commented Aug 17, 2024

An alternative to approving and merging this change could be to retroactively review and accept the changes for 3.13. I know we wouldn't normally apply such style-sync changes in an RC, but I also note that the AST doesn't change for the non-test code and changes only very modestly in tests. If @encukou would be okay with not applying this PR, that would be my preference.

@encukou
Copy link
Member

encukou commented Aug 19, 2024

I agree that, since the mistake is made, it's better to keep the style changes in than to revert them and re-introduce later.
@Yhg1s, do you agree?

@Yhg1s
Copy link
Member

Yhg1s commented Sep 2, 2024

Yes, the changes seem benign enough that there's no point in reverting and reapplying.

@Yhg1s Yhg1s closed this Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants