Add isort config.

.isort.cfg

@@ -0,0 +1,16 @@
+[settings]
+# # Needed for black compatibility
+multi_line_output=3
+include_trailing_comma=True
+force_grid_wrap=0
+line_length=88
+combine_as_imports=True
+
+# List sections with django and 
+known_django=django
+known_downloadview=django_downloadview
+
+sections=FUTURE,STDLIB,DJANGO,DOWNLOADVIEW,THIRDPARTY,FIRSTPARTY,LOCALFOLDER
+
+# If set, imports will be sorted within their section independent to the import_type.
+force_sort_within_sections=True

Makefile

@@ -104,4 +104,4 @@ black:
 
 .PHONY: isort
 isort:
-	$(ISORT) -m 3 --line-width=99 --use-parentheses --trailing-comma --recursive django_downloadview tests demo
+	$(ISORT) --recursive django_downloadview tests demo

demo/demoproject/apache/tests.py

@@ -1,7 +1,7 @@
 import os
 
-import django.test
 from django.core.files.base import ContentFile
+import django.test
 
 from django_downloadview.apache import assert_x_sendfile
 

demo/demoproject/http/tests.py

@@ -1,8 +1,9 @@
 import django.test
 
-from demoproject.compat import reverse
 from django_downloadview import assert_download_response
 
+from demoproject.compat import reverse
+
 
 class SimpleURLTestCase(django.test.TestCase):
     def test_download_response(self):

demo/demoproject/lighttpd/tests.py

@@ -1,7 +1,7 @@
 import os
 
-import django.test
 from django.core.files.base import ContentFile
+import django.test
 
 from django_downloadview.lighttpd import assert_x_sendfile
 

demo/demoproject/nginx/tests.py

@@ -1,7 +1,7 @@
 import os
 
-import django.test
 from django.core.files.base import ContentFile
+import django.test
 
 from django_downloadview.nginx import assert_x_accel_redirect
 

demo/demoproject/object/tests.py

@@ -1,9 +1,10 @@
-import django.test
 from django.core.files.base import ContentFile
+import django.test
+
+from django_downloadview import assert_download_response, temporary_media_root
 
 from demoproject.compat import reverse
 from demoproject.object.models import Document
-from django_downloadview import assert_download_response, temporary_media_root
 
 # Fixtures.
 slug = "hello-world"

demo/demoproject/object/views.py

@@ -1,6 +1,7 @@
-from demoproject.object.models import Document
 from django_downloadview import ObjectDownloadView
 
+from demoproject.object.models import Document
+
 #: Serve ``file`` attribute of ``Document`` model.
 default_file_view = ObjectDownloadView.as_view(model=Document)
 

demo/demoproject/path/tests.py

@@ -1,8 +1,9 @@
 import django.test
 
-from demoproject.compat import reverse
 from django_downloadview import assert_download_response
 
+from demoproject.compat import reverse
+
 
 class StaticPathTestCase(django.test.TestCase):
     def test_download_response(self):

demo/demoproject/settings.py

@@ -1,10 +1,9 @@
 """Django settings for django-downloadview demo project."""
-import os
 from distutils.version import StrictVersion
+import os
 
 from django.utils.version import get_version
 
-
 # Configure some relative directories.
 demoproject_dir = os.path.dirname(os.path.abspath(__file__))
 demo_dir = os.path.dirname(demoproject_dir)

demo/demoproject/storage/tests.py

@@ -1,18 +1,19 @@
 import datetime
 import unittest
 
-import django.test
 from django.core.files.base import ContentFile
 from django.http.response import HttpResponseNotModified
+import django.test
 
-from demoproject.compat import reverse
-from demoproject.storage import views
 from django_downloadview import (
     assert_download_response,
     setup_view,
     temporary_media_root,
 )
 
+from demoproject.compat import reverse
+from demoproject.storage import views
+
 # Fixtures.
 file_content = "Hello world!\n"
 

demo/demoproject/virtual/tests.py

@@ -1,8 +1,9 @@
 import django.test
 
-from demoproject.compat import reverse
 from django_downloadview import assert_download_response
 
+from demoproject.compat import reverse
+
 
 class TextTestCase(django.test.TestCase):
     def test_download_response(self):

demo/demoproject/virtual/views.py

@@ -1,7 +1,8 @@
+from io import StringIO
+
 from django.core.files.base import ContentFile
 
 from django_downloadview import TextIteratorIO, VirtualDownloadView, VirtualFile
-from io import StringIO
 
 
 class TextDownloadView(VirtualDownloadView):

demo/setup.py

@@ -1,5 +1,6 @@
 """Python packaging."""
 import os
+
 from setuptools import setup
 
 here = os.path.abspath(os.path.dirname(__file__))

django_downloadview/__init__.py

@@ -1,7 +1,7 @@
 """Serve files with Django and reverse proxies."""
-import pkg_resources
-
 from django_downloadview.api import *  # NoQA
 
+import pkg_resources
+
 #: Module version, as defined in PEP-0396.
 __version__ = pkg_resources.get_distribution(__package__.replace("-", "_")).version

django_downloadview/api.py

@@ -2,22 +2,24 @@
 """Declaration of API shortcuts."""
 from django_downloadview.files import HTTPFile, StorageFile, VirtualFile
 from django_downloadview.io import BytesIteratorIO, TextIteratorIO
-from django_downloadview.middlewares import BaseDownloadMiddleware
 from django_downloadview.middlewares import (
+    BaseDownloadMiddleware,
     DownloadDispatcherMiddleware,
     SmartDownloadMiddleware,
 )
-from django_downloadview.response import DownloadResponse
-from django_downloadview.response import ProxiedDownloadResponse
+from django_downloadview.response import DownloadResponse, ProxiedDownloadResponse
 from django_downloadview.shortcuts import sendfile
-from django_downloadview.test import assert_download_response
-from django_downloadview.test import setup_view, temporary_media_root
-from django_downloadview.views import PathDownloadView
+from django_downloadview.test import (
+    assert_download_response,
+    setup_view,
+    temporary_media_root,
+)
 from django_downloadview.views import (
     BaseDownloadView,
     DownloadMixin,
     HTTPDownloadView,
     ObjectDownloadView,
+    PathDownloadView,
     StorageDownloadView,
     VirtualDownloadView,
 )

django_downloadview/files.py

@@ -1,12 +1,13 @@
 """File wrappers for use as exchange data between views and responses."""
+from io import BytesIO
 from urllib.parse import urlparse
 
-import requests
 from django.core.files.base import File
 from django.utils.encoding import force_bytes
 
 from django_downloadview.io import BytesIteratorIO
-from io import BytesIO
+
+import requests
 
 
 class StorageFile(File):

django_downloadview/io.py

@@ -1,8 +1,8 @@
 """Low-level IO operations, for use with file wrappers."""
-from django.utils.encoding import force_bytes, force_text
-
 import io
 
+from django.utils.encoding import force_bytes, force_text
+
 
 class TextIteratorIO(io.TextIOBase):
     """A dynamically generated TextIO-like object.

django_downloadview/views/http.py

@@ -1,9 +1,9 @@
 """Stream files given an URL, i.e. files you want to proxy."""
-import requests
-
 from django_downloadview.files import HTTPFile
 from django_downloadview.views.base import BaseDownloadView
 
+import requests
+
 
 class HTTPDownloadView(BaseDownloadView):
     """Proxy files that live on remote servers."""

tests/api.py

@@ -1,10 +1,10 @@
 """Test suite around :mod:`django_downloadview.api` and deprecation plan."""
+from importlib import import_module, reload
 import unittest
 import warnings
-from importlib import import_module, reload
 
-import django.test
 from django.core.exceptions import ImproperlyConfigured
+import django.test
 from django.test.utils import override_settings
 
 

tests/sendfile.py

@@ -1,6 +1,6 @@
 """Tests around :py:mod:`django_downloadview.sendfile`."""
-import django.test
 from django.http import Http404
+import django.test
 
 from django_downloadview.response import DownloadResponse
 from django_downloadview.shortcuts import sendfile

tests/views.py

@@ -1,14 +1,14 @@
 """Tests around :mod:`django_downloadview.views`."""
 import calendar
+from datetime import datetime
 import os
 import unittest
-from datetime import datetime
 from unittest import mock
 
-import django.test
 from django.core.files import File
 from django.http import Http404
 from django.http.response import HttpResponseNotModified
+import django.test
 
 from django_downloadview import exceptions, views
 from django_downloadview.test import setup_view

tox.ini

@@ -55,5 +55,5 @@ commands =
     fullrelease
 
 [flake8]
-max-line-length = 99
+max-line-length = 88
 ignore = E203, W503