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 implicit namespaces #126

Merged
merged 4 commits into from
Dec 28, 2023
Merged

Support implicit namespaces #126

merged 4 commits into from
Dec 28, 2023

Conversation

gforcada
Copy link
Collaborator

Closes #120

Seems that all the discussion we had with @ericof at the issue is not so relevant, as we are already using the .egg-info information to know where the source code is located 🎉

Since Python 3.3 one does not need to create `__init__.py` files to mark
a folder to be part of a namespace, but the simple presence of a python
file deep inside it is enough to create it.

Moreover, since `setuptools` 62.3.0 it is officially discouraged to use
the old `pkg_resources` namespace technique.
Copy link
Owner

@reinout reinout left a comment

Choose a reason for hiding this comment

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

Looks handy!

z3c/dependencychecker/modules.py Show resolved Hide resolved
z3c/dependencychecker/modules.py Show resolved Hide resolved
@gforcada
Copy link
Collaborator Author

May I merge and release then? 😄 I'm never comfortable doing that without asking and getting approval 😅

But given that plenty of packages are configured with plone/meta which includes in one of its tox.ini/GHA/GitLab CI a check with z3c.dependencychecker, we need to get this PR released, otherwise we block any migration to implicit namespaces ✨

@reinout
Copy link
Owner

reinout commented Dec 27, 2023 via email

@gforcada gforcada merged commit d1d0c00 into master Dec 28, 2023
7 checks passed
@gforcada gforcada deleted the support-implicit-namespaces branch December 28, 2023 08:32
@gforcada
Copy link
Collaborator Author

z3c.dependencychecker==2.14 is out! 🎉

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.

Support native namespace packages
2 participants