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

Support Python 3.12 #165

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Support Python 3.12 #165

merged 1 commit into from
Nov 17, 2023

Conversation

max-muoto
Copy link
Contributor

@max-muoto max-muoto commented Nov 11, 2023

This PR adds support up to Python 3.12. pkg_resources is no longer accessible by default in Python 3.12, which means this library is incompatible with Python 3.12 without setuptools installed. This PR moves from using pkg_resources to importlib.resources which is the intended way of replacing the resources API of pgk_resources. See here.

@max-muoto
Copy link
Contributor Author

@kaisen Any chance you might be able to take a look here?

@kaisen
Copy link
Member

kaisen commented Nov 13, 2023

Thanks for the PR @max-muoto! The PR looks good to me but I'll defer to @ymilki or @benbariteau for the final shipit.

@max-muoto
Copy link
Contributor Author

Following up here @ymilki, @benbariteau!

Copy link
Member

@ymilki ymilki left a comment

Choose a reason for hiding this comment

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

lgtm

@ymilki ymilki merged commit 2e54d8a into Yelp:master Nov 17, 2023
@ymilki ymilki mentioned this pull request Nov 17, 2023
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.

3 participants