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

Revert "Use importlib instead of pkg_resources for determining namespace packages" #1409

Merged
merged 2 commits into from
Feb 27, 2022

Conversation

DanielNoord
Copy link
Collaborator

Reverts #1326

See discussion in #1406.

Sad 😢

@DanielNoord DanielNoord added the Maintenance Discussion or action around maintaining astroid or the dev workflow label Feb 27, 2022
@DanielNoord DanielNoord added this to the 2.10.0 milestone Feb 27, 2022
astroid/const.py Outdated Show resolved Hide resolved
Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

Getting rid of pkg_resourceswon't be easy, but a lot of preliminary work was already done and the situation is still better than in 2.9, thank you @DanielNoord 👍

@DanielNoord
Copy link
Collaborator Author

Getting rid of pkg_resourceswon't be easy, but a lot of preliminary work was already done and the situation is still better than in 2.9, thank you @DanielNoord 👍

With some debug tips from @cdce8p I'm certain we can figure this out. I know much more about importlib now so we're probably on the right track already 😄

Copy link
Member

@cdce8p cdce8p left a comment

Choose a reason for hiding this comment

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

LGTM. I left some debug tips here.

@cdce8p cdce8p merged commit a5bd030 into main Feb 27, 2022
@cdce8p cdce8p deleted the revert-1326-pkg_resources branch February 27, 2022 12:19
@cdce8p cdce8p mentioned this pull request Feb 27, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Discussion or action around maintaining astroid or the dev workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants