Skip to content

Conversation

@ryanwilsonperkin
Copy link
Contributor

With the current use of find_packages, the "tests" package is discovered and installed alongside the django_sharding and django_sharding_packages. This appears to be unintentional, and we probably don't want users to accidentally get this test package.

This change excludes it from being discovered during packaging.

With the current use of find_packages, the "tests" package is discovered
and installed alongside the django_sharding and
django_sharding_packages. This appears to be unintentional, and we
probably don't want users to accidentally get this test package.

This change excludes it from being discovered during packaging.
@coveralls
Copy link

coveralls commented May 4, 2018

Coverage Status

Coverage remained the same at 89.899% when pulling c87c461 on ryanwilsonperkin:remove-tests-from-install into 86d9f56 on JBKahn:master.

@rafis
Copy link

rafis commented Jul 8, 2018

This is true, please merge this pull request!

@JBKahn
Copy link
Owner

JBKahn commented Jul 8, 2018

Ok I'll get on it this week

@JBKahn JBKahn merged commit 97900e4 into JBKahn:master Jul 16, 2018
@JBKahn JBKahn mentioned this pull request Jul 16, 2018
JBKahn added a commit that referenced this pull request Jul 16, 2018
small change, includes: #78
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

Successfully merging this pull request may close these issues.

4 participants