Hi! I package news-updater for Arch Linux.
I realized, that the package installs the tests top-level to site-packages, due to find_packages() being called without any parameters:
This is bound to lead to problems with other packages doing the same.
It's usually better to not include the tests in the bdist/wheel package (but of course in the sdist package!), unless they are situated below the main packages scope (e.g. <my_package>/tests).
Hi! I package news-updater for Arch Linux. I realized, that the package installs the tests top-level to site-packages, due to
find_packages()being called without any parameters:https://github.com/nextcloud/news-updater/blob/955b021bb31b0d44295080c6f669ca4536d0a8e5/setup.py#L24
This is bound to lead to problems with other packages doing the same. It's usually better to not include the tests in the bdist/wheel package (but of course in the sdist package!), unless they are situated below the main packages scope (e.g.
<my_package>/tests).