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

Add Python 3.12 compatibility, drop Python 3.7 (EOL) #36

Merged
merged 2 commits into from
Oct 15, 2023

Conversation

zurcher
Copy link
Contributor

@zurcher zurcher commented Oct 12, 2023

This solves the following error when installing from source with pip install -e:

AttributeError: module 'pkgutil' has no attribute 'ImpImporter'.

Tested with Python 3.12 and 3.10.11.

@zurcher zurcher force-pushed the setuptools branch 6 times, most recently from e1fd707 to 771b7e5 Compare October 13, 2023 00:04
@zurcher zurcher changed the title Update setuptools version for Python 3.12 compatibility Add Python 3.12 compatibility, drop Python 3.7 (EOL) Oct 13, 2023
@zurcher
Copy link
Contributor Author

zurcher commented Oct 13, 2023

Python 3.7 is EOL:
https://devguide.python.org/versions/

@zurcher zurcher marked this pull request as draft October 13, 2023 17:33
@nathanhi
Copy link
Owner

nathanhi commented Oct 15, 2023

Thank you once again! 🥳 Sorry for the delay releasing the upcoming package!

@nathanhi nathanhi marked this pull request as ready for review October 15, 2023 08:49
nathanhi
nathanhi previously approved these changes Oct 15, 2023
@nathanhi
Copy link
Owner

nathanhi commented Oct 15, 2023

Seems like the tests are failing in Python 3.12 due to the usage of deprecated / removed test methods in PyFilesystem2:

PyFilesystem/pyfilesystem2#568

I've added a workaround until this gets fixed upstream.

@nathanhi nathanhi merged commit a4ec954 into nathanhi:master Oct 15, 2023
17 checks passed
@zurcher
Copy link
Contributor Author

zurcher commented Oct 16, 2023

Thanks for cleaning this up!

@nathanhi nathanhi added bug Something isn't working enhancement New feature or request labels Oct 16, 2023
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants