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

Use native importlib.metadata when possible. #578

Closed
wants to merge 1 commit into from

Conversation

kousu
Copy link

@kousu kousu commented Jun 2, 2022

The importlib_metadata backport is only needed for python 3.7 now, since python 3.6 is at End-of-Life, but keyring makes it, and its dependency zipp get downloaded and installed.

The importlib_metadata backport is only needed for python 3.7 now,
but it, and its dependency zipp, still get downloaded and installed
for everyone who uses keyring.
@mitya57
Copy link
Collaborator

mitya57 commented Jun 3, 2022

Hi!

This is third pull request for importlib.metadata change. We have already had #576 and #577.

#577 is approved, I hope we will merge it soon. Note that we need to use importlib.metadata only for Python ≥ 3.10, not ≥ 3.8, because 3.8 and 3.9 don't support group argument (see the failed tests).

So closing this in favor of #577.

@mitya57 mitya57 closed this Jun 3, 2022
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.

None yet

2 participants