Skip to content

Update for python 3.10+: branch on SelectableGroups #41

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

Conversation

adrianeboyd
Copy link
Contributor

  • Branch on presence of SelectableGroups when retrieving entry points
  • Branch on python version in tests when creating dummy entry points

- Branch on presence of `SelectableGroups` when retrieving entry points
- Branch on python version in tests when creating dummy entry points
@adrianeboyd
Copy link
Contributor Author

This still needs some more testing beyond the test suite...

@adrianeboyd
Copy link
Contributor Author

I've tested this with thinc and spacy with a range of python versions including python 3.11.0b4 without running into any problems.

@rmitsch rmitsch self-requested a review July 15, 2022 09:06
@adrianeboyd adrianeboyd linked an issue Jul 15, 2022 that may be closed by this pull request
Copy link
Collaborator

@rmitsch rmitsch left a comment

Choose a reason for hiding this comment

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

I agree that this is a better approach if we want to abstain from re-introducing importlib_metadata as dependency (misunderstood that we'd rather drop as much vendored stuff as possible).

@adrianeboyd adrianeboyd merged commit 75f5e9c into explosion:master Jul 26, 2022
adrianeboyd added a commit to adrianeboyd/catalogue that referenced this pull request Oct 4, 2022
adrianeboyd added a commit to adrianeboyd/catalogue that referenced this pull request Oct 4, 2022
adrianeboyd added a commit to adrianeboyd/catalogue that referenced this pull request Oct 4, 2022
adrianeboyd added a commit that referenced this pull request Oct 7, 2022
* Move catalogue.py to module dir

* Update setup

* Add vendored importlib_metadata v3.2.0

* Move tests into package

* Backport changes from #41

* Check for select rather than to-be-deprecated SelectableGroups

* Set version to v1.0.1, update classifiers

* Update CI
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.

Importlib issue with python3.10
3 participants