search

miracle2k / django-assets

Django webassets integration.
BSD 2-Clause "Simplified" License
89 stars 79 forks source link

Don't include 'tests' in setup.py #53

Closed adamchainz closed 9 years ago

adamchainz commented 9 years ago

Installing django-assets adds tests in site-packages since find_packages is finding it and shipping it to PyPi. Found this with some code doing relative imports.

Tested the find_packages change in python shell.

adamchainz commented 9 years ago

This is what I can currently do with the latest django-assets v 0.10 on my django project...

$ ipython
In [1]: import tests

In [2]: tests??
Type:        module
String form: <module 'tests' from '/opt/yplan/venv/local/lib/python2.7/site-packages/tests/__init__.pyc'>
File:        /opt/yplan/venv/local/lib/python2.7/site-packages/tests/__init__.py
Source:
from nose import SkipTest
try:
    from django.conf import settings
except ImportError:
    raise SkipTest()

# Setup a Django environment, before we do anything else.
#
# Most Django imports fail one way or another without an
# environment.
#
# We can't even use setup_package() here, because then
# module-global imports in our test submodules still run
# first.

from django.conf import settings
settings.configure(INSTALLED_APPS=['django_assets'])

:scream:

adamchainz commented 9 years ago

:smile_cat: