Removed unused imports and optimize imports

Author: luzfcb

Diff for: blogs/admin.py

@@ -1,7 +1,11 @@
 from django.contrib import admin
 from django.core.management import call_command
 
-from .models import BlogEntry, Feed, FeedAggregate
+from .models import (
+    BlogEntry,
+    Feed,
+    FeedAggregate,
+)
 
 
 @admin.register(BlogEntry)

Diff for: blogs/management/commands/update_blogs.py

@@ -1,8 +1,15 @@
 from django.core.management.base import BaseCommand
 from django.utils.timezone import now
 
-from ...models import BlogEntry, RelatedBlog, Feed
-from ...parser import get_all_entries, update_blog_supernav
+from ...models import (
+    BlogEntry,
+    Feed,
+    RelatedBlog,
+)
+from ...parser import (
+    get_all_entries,
+    update_blog_supernav,
+)
 
 
 class Command(BaseCommand):

Diff for: blogs/models.py

@@ -1,8 +1,6 @@
 import feedparser
-
 from bs4 import BeautifulSoup
 from bs4.element import Comment
-
 from django.db import models
 
 from cms.models import ContentManageable

Diff for: blogs/parser.py

@@ -1,12 +1,18 @@
 import datetime
-import feedparser
 
+import feedparser
 from django.conf import settings
 from django.template.loader import render_to_string
-from django.utils.timezone import make_aware, utc
+from django.utils.timezone import (
+    make_aware,
+    utc,
+)
 
 from boxes.models import Box
-from .models import BlogEntry, Feed
+from .models import (
+    BlogEntry,
+    Feed,
+)
 
 
 def get_all_entries(feed_url):

Diff for: blogs/tests/test_models.py

@@ -1,7 +1,10 @@
 from django.test import TestCase
 from django.utils import timezone
 
-from ..models import BlogEntry, Feed
+from ..models import (
+    BlogEntry,
+    Feed,
+)
 
 
 class BlogModelTest(TestCase):

Diff for: blogs/tests/test_parser.py

@@ -1,8 +1,8 @@
 import datetime
 import unittest
 
-from ..parser import get_all_entries
 from .utils import get_test_rss_path
+from ..parser import get_all_entries
 
 
 class BlogParserTest(unittest.TestCase):

Diff for: blogs/tests/test_templatetags.py

@@ -1,11 +1,18 @@
 from django.core.management import call_command
+from django.template import (
+    Context,
+    Template,
+)
 from django.test import TestCase
-from django.template import Template, Context
 from django.utils.timezone import now
 
-from ..templatetags.blogs import get_latest_blog_entries
-from ..models import BlogEntry, Feed, FeedAggregate
 from .utils import get_test_rss_path
+from ..models import (
+    BlogEntry,
+    Feed,
+    FeedAggregate,
+)
+from ..templatetags.blogs import get_latest_blog_entries
 
 
 class BlogTemplateTagTest(TestCase):

Diff for: blogs/tests/test_views.py

@@ -1,10 +1,12 @@
 from django.core.management import call_command
-from django.urls import reverse
 from django.test import TestCase
-
-from ..models import BlogEntry, Feed
+from django.urls import reverse
 
 from .utils import get_test_rss_path
+from ..models import (
+    BlogEntry,
+    Feed,
+)
 
 
 class BlogViewTest(TestCase):

Diff for: blogs/urls.py

@@ -1,6 +1,7 @@
-from . import views
 from django.urls import path
 
+from . import views
+
 urlpatterns = [
     path('', views.BlogHome.as_view(), name='blog'),
 ]

Diff for: boxes/admin.py

@@ -1,4 +1,5 @@
 from django.contrib import admin
+
 from cms.admin import ContentManageableModelAdmin
 from .models import Box
 

Diff for: boxes/factories.py

@@ -2,13 +2,11 @@
 import pathlib
 
 import factory
-
 from django.conf import settings
 from factory.django import DjangoModelFactory
 
-from .models import Box
-
 from users.factories import UserFactory
+from .models import Box
 
 
 class BoxFactory(DjangoModelFactory):

Diff for: boxes/models.py

@@ -11,6 +11,7 @@
 from django.conf import settings
 from django.db import models
 from markupfield.fields import MarkupField
+
 from cms.models import ContentManageable
 
 DEFAULT_MARKUP_TYPE = getattr(settings, 'DEFAULT_MARKUP_TYPE', 'restructuredtext')

Diff for: boxes/tests.py

@@ -1,6 +1,11 @@
 import logging
+
 from django import template
-from django.test import TestCase, override_settings
+from django.test import (
+    TestCase,
+    override_settings,
+)
+
 from .models import Box
 
 logging.disable(logging.CRITICAL)

Diff for: boxes/urls.py

@@ -1,6 +1,7 @@
-from .views import box
 from django.urls import path
 
+from .views import box
+
 urlpatterns = [
     path('<slug:label>/', box, name='box'),
 ]

Diff for: boxes/views.py

@@ -1,5 +1,6 @@
 from django.http import HttpResponse
 from django.shortcuts import get_object_or_404
+
 from .models import Box
 
 

Diff for: cms/management/commands/create_initial_data.py

@@ -3,7 +3,10 @@
 import pprint
 
 from django.apps import apps
-from django.core.management import BaseCommand, call_command
+from django.core.management import (
+    BaseCommand,
+    call_command,
+)
 
 
 class Command(BaseCommand):

Diff for: cms/tests.py

@@ -1,12 +1,15 @@
+import datetime
 import unittest
 from unittest import mock
 
-from django.template import Template, Context
+from django.template import (
+    Context,
+    Template,
+)
 from django.test import TestCase
 
 from .admin import ContentManageableModelAdmin
 from .views import legacy_path
-import datetime
 
 
 class ContentManageableAdminTests(unittest.TestCase):

Diff for: cms/views.py

@@ -1,7 +1,8 @@
-from django.urls import reverse
-from django.shortcuts import render
 from urllib.parse import urljoin
 
+from django.shortcuts import render
+from django.urls import reverse
+
 LEGACY_PYTHON_DOMAIN = 'http://legacy.python.org'
 PYPI_URL = 'https://pypi.org/'
 

Diff for: codesamples/admin.py

@@ -1,7 +1,6 @@
 from django.contrib import admin
 
-from .models import CodeSample
 from cms.admin import ContentManageableModelAdmin
-
+from .models import CodeSample
 
 admin.site.register(CodeSample, ContentManageableModelAdmin)

Diff for: codesamples/factories.py

@@ -3,9 +3,8 @@
 import factory
 from factory.django import DjangoModelFactory
 
-from .models import CodeSample
-
 from users.factories import UserFactory
+from .models import CodeSample
 
 
 class CodeSampleFactory(DjangoModelFactory):

Diff for: codesamples/models.py

@@ -1,13 +1,14 @@
 from django.conf import settings
 from django.db import models
-from django.template.defaultfilters import truncatechars, striptags
+from django.template.defaultfilters import (
+    striptags,
+    truncatechars,
+)
 from markupfield.fields import MarkupField
 
 from cms.models import ContentManageable
-
 from .managers import CodeSampleQuerySet
 
-
 DEFAULT_MARKUP_TYPE = getattr(settings, 'DEFAULT_MARKUP_TYPE', 'html')
 
 

Diff for: community/admin.py

@@ -1,7 +1,15 @@
 from django.contrib import admin
 
-from .models import Link, Photo, Post, Video
-from cms.admin import ContentManageableModelAdmin, ContentManageableStackedInline
+from cms.admin import (
+    ContentManageableModelAdmin,
+    ContentManageableStackedInline,
+)
+from .models import (
+    Link,
+    Photo,
+    Post,
+    Video,
+)
 
 
 class LinkInline(ContentManageableStackedInline):

Diff for: community/models.py

@@ -1,15 +1,12 @@
 from django.contrib.postgres.fields import JSONField
-from django.urls import reverse
 from django.db import models
+from django.urls import reverse
 from django.utils.translation import gettext_lazy as _
-
 from markupfield.fields import MarkupField
 
 from cms.models import ContentManageable
-
 from .managers import PostQuerySet
 
-
 DEFAULT_MARKUP_TYPE = 'html'
 
 

Diff for: community/urls.py

@@ -1,6 +1,7 @@
-from . import views
 from django.urls import path
 
+from . import views
+
 app_name = 'community'
 urlpatterns = [
     path('', views.PostList.as_view(), name='post_list'),

Diff for: community/views.py

@@ -1,4 +1,7 @@
-from django.views.generic import ListView, DetailView
+from django.views.generic import (
+    DetailView,
+    ListView,
+)
 
 from .models import Post
 

Diff for: companies/admin.py

@@ -1,7 +1,6 @@
 from django.contrib import admin
 
 from cms.admin import NameSlugAdmin
-
 from .models import Company
 
 

Diff for: companies/models.py

@@ -5,7 +5,6 @@
 
 from cms.models import NameSlugModel
 
-
 DEFAULT_MARKUP_TYPE = getattr(settings, 'DEFAULT_MARKUP_TYPE', 'restructuredtext')
 
 

Diff for: companies/tests.py

@@ -1,6 +1,6 @@
 from django.test import TestCase
 
-from . import admin     # coverage FTW
+from . import admin  # noqa: F401 coverage FTW
 from .templatetags.companies import render_email
 
 

Diff for: custom_storages/storage.py

@@ -1,14 +1,18 @@
 import os
 import posixpath
 import re
-
-from urllib.parse import unquote, urldefrag
+from urllib.parse import (
+    unquote,
+    urldefrag,
+)
 
 from django.conf import settings
-from django.contrib.staticfiles.storage import ManifestFilesMixin, StaticFilesStorage
+from django.contrib.staticfiles.storage import (
+    ManifestFilesMixin,
+    StaticFilesStorage,
+)
 from django.contrib.staticfiles.utils import matches_patterns
 from django.core.files.base import ContentFile
-
 from pipeline.storage import PipelineMixin
 from storages.backends.s3boto3 import S3Boto3Storage
 

Diff for: docs/source/conf.py

@@ -1,7 +1,5 @@
 #!/usr/bin/env python3
 
-import sys
-import os
 import time
 
 extensions = [

Diff for: downloads/admin.py

@@ -1,7 +1,14 @@
 from django.contrib import admin
 
-from .models import OS, Release, ReleaseFile
-from cms.admin import ContentManageableModelAdmin, ContentManageableStackedInline
+from cms.admin import (
+    ContentManageableModelAdmin,
+    ContentManageableStackedInline,
+)
+from .models import (
+    OS,
+    Release,
+    ReleaseFile,
+)
 
 
 @admin.register(OS)