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

Prep removal of importlib.resources.open_text. #12462

Merged
merged 3 commits into from
Mar 26, 2024

Conversation

jsirois
Copy link
Contributor

@jsirois jsirois commented Jan 5, 2024

The importlib.resources.open_text was deprecated in Python 3.11 and is
removed in 3.13.

The `importlib.resources.open_text` was deprecated in Python 3.11 and is
removed in 3.13.
@jsirois
Copy link
Contributor Author

jsirois commented Jan 5, 2024

FWICT there is CI bitrot over the last ~5 days. This PR has similar small footprint / scope and similar failures for example: https://github.com/pypa/pip/actions/runs/7379405369?pr=12457

@pradyunsg am I off here with respect to CI / CI state?

@jsirois
Copy link
Contributor Author

jsirois commented Jan 5, 2024

Ah, perhaps #12458 fixes the CI issues here.

@pradyunsg
Copy link
Member

Yea, we need to fix the CI at some point soon. I'm trying to avoid doing too much OSS stuff while I'm with family, but I expect I'll spend time on the CI situation when I get back from vacation.

@jsirois
Copy link
Contributor Author

jsirois commented Feb 5, 2024

Alright @pradyunsg, I merged in @sbidoul's CI fixes, did a typing fix of my own for this PR and things look good now.

@uranusjr uranusjr merged commit 0807480 into pypa:main Mar 26, 2024
26 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 11, 2024
@jsirois jsirois deleted the 3.13/debug/fix branch April 27, 2024 16:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants