-
Notifications
You must be signed in to change notification settings - Fork 171
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
pyproject include = ["LICENSE", "CONTRIBUTING.rst"...] #324
Labels
Comments
There hasn't been any activity on this issue recently. This issue has been automatically marked as stale because of that. It will be closed if no further activity occurs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is a question.
In the
pyproject.toml
there is:I haven't done much Python packaging. But for other packages that I've done in Debian: this was never there, and because it's there I need to fix it in the Debian package (else the files go to an unexpected place).
If I install python-ring-doorbell with pip I see:
Which does not seem the correct place for CONTRIBUTING.rst (same for the other files).
I suggest to delete the include (I can do a quick PR if needed) to avoid installing those files in
site-packages/
(this is using pip), besides making Debian package more standard. Of course, unless there is a good reason to have theinclude =
there (which I might have missed, only seeing what's happening in other packages).Also, I don't know the best way to include the files in the pip (I can have a look at other packages...) or if they should be in a subdirectory.
Thanks for considering it!
The text was updated successfully, but these errors were encountered: