-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Migrate to pyproject toml #3063
Conversation
setuptools already exclude them by default in the flat-layout setup, see https://setuptools.pypa.io/en/latest/userguide/package_discovery.html\#flat-layout
@benoitc can I have a CI run please 🙂 ? |
d065fb4
to
fdd23e8
Compare
Thanks for the patch. can you describe what's supported in this PR? |
Hello Benoit.
I am available if you have questions. |
Hi thanks for your comment. I'm not a fan such file format, but let's follow the trend :) Thanks for the PR! |
Thank you for the merge 🙂 |
Arg, I see an error slept in: the diff --git a/pyproject.toml b/pyproject.toml
index ec64a328..f4afa721 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -71,7 +71,7 @@ addopts = "--assert=plain --cov=gunicorn --cov-report=xml"
[tool.setuptools]
zip-safe = false
include-package-data = true
-license-files = ["['LICENSE']"]
+license-files = ["LICENSE"]
[tool.setuptools.packages]
find = {namespaces = false} Or I can make a PR if you want. |
Link in commit message was broken, so adding intended one back here in case someone reviews this while working on Adding references for future readers: |
It's 2023 and the future is pyproject.toml 🎉