You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
include = [
"LICENSE",
"CONTRIBUTING.rst",
"CHANGELOG.rst",
"scripts/ringcli.py"
]
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).
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 the include = 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:
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.
Please make sure to update to the latest ring_doorbell version and check if that solves the issue.
Thank you for your contributions.
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: