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

Wheel dist includes the tests package #355

Closed
ionelmc opened this issue May 18, 2021 · 2 comments
Closed

Wheel dist includes the tests package #355

ionelmc opened this issue May 18, 2021 · 2 comments

Comments

@ionelmc
Copy link

ionelmc commented May 18, 2021

This breaks my application's tests since the tests package in site-package will take precedence.

This regression appeared in 3.3 (3.2 is fine). Looks like the cause is this change: 9175803#diff-41d5a52589e0480be9c099d2bba7a8135b8b0d71bcbb8df3582a8df1c2295003

Looks like the intended change there was to include the test in the sdist. A change to find_packages to use exclude or include is also needed.

Similar problem to 3YOURMIND/django-migration-linter#147

@hsmett
Copy link

hsmett commented Apr 6, 2022

Hello,
Just a quick note to let you know that this issue is still there.
The 'tests' directory packaged with the application is installed in 'site-packages/tests'.
Thank you

@claudep
Copy link
Contributor

claudep commented Apr 6, 2022

Patch welcome :-)

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

No branches or pull requests

3 participants