Results of two runs of pre-commit run --all -- still fails due to "prospector"

docs/_static/css/custom.css

@@ -20,4 +20,4 @@
   text-rendering: auto;
   -webkit-font-smoothing: antialiased;
 
-}
+}

media/css/header.css

@@ -40,4 +40,3 @@
 .ui-autocomplete { background: #fff; text-align: left; border: 1px solid #bfbfbf; }
 .ui-state-hover { background: #eee; }
 .ui-menu-item a { display: block; padding: 3px 10px; }
-

media/javascript/searchtools.js

@@ -512,4 +512,4 @@ var Search = {
 
 $(document).ready(function() {
   Search.init();
-});
+});

readthedocs/analytics/migrations/0001_initial.py

@@ -1,8 +1,9 @@
 # Generated by Django 2.2.12 on 2020-05-19 00:45
 
 import datetime
-from django.db import migrations, models
+
 import django.db.models.deletion
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/api/v3/proxied_views.py

@@ -1,6 +1,6 @@
+from readthedocs.core.mixins import ProxiedAPIMixin
 from readthedocs.core.utils.extend import SettingsOverrideObject
 from readthedocs.embed.v3.views import EmbedAPIBase
-from readthedocs.core.mixins import ProxiedAPIMixin
 
 
 class ProxiedEmbedAPIBase(ProxiedAPIMixin, EmbedAPIBase):

readthedocs/api/v3/tests/test_environmentvariables.py

@@ -1,9 +1,10 @@
-from .mixins import APIEndpointMixin
+import django_dynamic_fixture as fixture
 from django.urls import reverse
 
-import django_dynamic_fixture as fixture
 from readthedocs.projects.models import EnvironmentVariable
 
+from .mixins import APIEndpointMixin
+
 
 class EnvironmentVariablessEndpointTests(APIEndpointMixin):
     def setUp(self):

readthedocs/api/v3/tests/test_remoteorganizations.py

@@ -1,13 +1,10 @@
-from django.urls import reverse
-
-from allauth.socialaccount.models import SocialAccount
 import django_dynamic_fixture as fixture
+from allauth.socialaccount.models import SocialAccount
+from django.urls import reverse
 
 from readthedocs.oauth.constants import GITHUB
-from readthedocs.oauth.models import (
-    RemoteOrganization,
-    RemoteOrganizationRelation,
-)
+from readthedocs.oauth.models import RemoteOrganization, RemoteOrganizationRelation
+
 from .mixins import APIEndpointMixin
 
 

readthedocs/api/v3/tests/test_remoterepositories.py

@@ -1,7 +1,6 @@
-from django.urls import reverse
-
-from allauth.socialaccount.models import SocialAccount
 import django_dynamic_fixture as fixture
+from allauth.socialaccount.models import SocialAccount
+from django.urls import reverse
 
 from readthedocs.oauth.constants import GITHUB
 from readthedocs.oauth.models import (
@@ -11,6 +10,7 @@
     RemoteRepositoryRelation,
 )
 from readthedocs.projects.constants import REPO_TYPE_GIT
+
 from .mixins import APIEndpointMixin
 
 

readthedocs/audit/apps.py

@@ -1,7 +1,6 @@
 """Audit module."""
 
 import structlog
-
 from django.apps import AppConfig
 
 log = structlog.get_logger(__name__)

readthedocs/audit/migrations/0001_initial.py

@@ -1,9 +1,9 @@
 # Generated by Django 2.2.24 on 2021-07-29 18:34
 
-from django.conf import settings
-from django.db import migrations, models
 import django.db.models.deletion
 import django_extensions.db.fields
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/audit/migrations/0002_add_organization.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.24 on 2021-09-14 20:53
 
-from django.db import migrations, models
 import django.db.models.deletion
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/audit/signals.py

@@ -6,8 +6,8 @@
     user_logged_out,
     user_login_failed,
 )
-from django.dispatch import receiver
 from django.db.models import Q
+from django.dispatch import receiver
 
 from readthedocs.audit.models import AuditLog
 

readthedocs/builds/migrations/0022_migrate_protected_versions.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.12 on 2020-05-18 20:06
 
-from django.db import migrations
 from django.conf import settings
+from django.db import migrations
 
 
 def forwards_func(apps, schema_editor):

readthedocs/builds/migrations/0025_migrate_private_versions.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.12 on 2020-06-11 01:14
 
-from django.db import migrations
 from django.conf import settings
+from django.db import migrations
 
 
 def forwards_func(apps, schema_editor):

readthedocs/builds/migrations/0029_add_time_fields.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.16 on 2020-11-18 16:26
 
-from django.db import migrations
 import django_extensions.db.fields
+from django.db import migrations
 
 
 class Migration(migrations.Migration):

readthedocs/builds/migrations/0030_add_automation_rule_matches.py

@@ -1,8 +1,8 @@
 # Generated by Django 2.2.16 on 2020-11-10 22:42
 
-from django.db import migrations, models
 import django.db.models.deletion
 import django_extensions.db.fields
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/builds/migrations/0033_dont_cascade_delete_builds.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.17 on 2020-11-25 16:01
 
-from django.db import migrations, models
 import django.db.models.deletion
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/builds/signals.py

@@ -2,7 +2,6 @@
 
 import django.dispatch
 
-
 build_complete = django.dispatch.Signal()
 # Useful to know when to purge the footer
 version_changed = django.dispatch.Signal()

readthedocs/builds/urls.py

@@ -3,7 +3,6 @@
 from django.urls import re_path
 from django.views.generic.base import RedirectView
 
-
 urlpatterns = [
     re_path(
         r"^(?P<project_slug>[-\w]+)/(?P<build_pk>\d+)/$",

readthedocs/core/history.py

@@ -1,6 +1,6 @@
-import structlog
 from functools import partial
 
+import structlog
 from django import forms
 from django.db import models
 from django.utils.translation import gettext_lazy as _

readthedocs/core/management/commands/collectstatic.py

@@ -5,7 +5,8 @@
 """
 
 from django.contrib.staticfiles.management.commands import collectstatic
-from readthedocs.core.signals import pre_collectstatic, post_collectstatic
+
+from readthedocs.core.signals import post_collectstatic, pre_collectstatic
 
 
 class Command(collectstatic.Command):

readthedocs/core/migrations/0007_historicaluser.py

@@ -1,11 +1,11 @@
 # Generated by Django 2.2.24 on 2021-07-21 15:36
 
-from django.conf import settings
 import django.contrib.auth.validators
-from django.db import migrations, models
 import django.db.models.deletion
 import django.utils.timezone
 import simple_history.models
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/core/migrations/0009_historicaluserprofile.py

@@ -1,9 +1,9 @@
 # Generated by Django 2.2.24 on 2021-11-11 18:20
 
-from django.conf import settings
-from django.db import migrations, models
 import django.db.models.deletion
 import simple_history.models
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/core/migrations/0010_add_time_fields.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.24 on 2021-11-11 19:08
 
-from django.db import migrations
 import django_extensions.db.fields
+from django.db import migrations
 
 
 class Migration(migrations.Migration):

readthedocs/core/utils/contact.py

@@ -1,11 +1,9 @@
-import structlog
-
 import markdown
+import structlog
 from django.conf import settings
 from django.core.mail import send_mail
 from django.template import Context, Engine
 
-
 log = structlog.get_logger(__name__)
 
 

readthedocs/doc_builder/loader.py

@@ -3,7 +3,6 @@
 
 from django.conf import settings
 
-
 # Managers
 mkdocs = import_module(settings.MKDOCS_BACKEND)
 sphinx = import_module(settings.SPHINX_BACKEND)

readthedocs/embed/v3/tests/conftest.py

@@ -1,5 +1,6 @@
 import os
 import shutil
+
 import pytest
 
 from .utils import srcdir

readthedocs/oauth/management/commands/reconnect_remoterepositories.py

@@ -1,13 +1,13 @@
 import json
 
-from django.db.models import Q, Subquery
 from django.core.management.base import BaseCommand
+from django.db.models import Q, Subquery
 
 from readthedocs.oauth.models import RemoteRepository
 from readthedocs.oauth.services import registry
 from readthedocs.oauth.services.base import SyncServiceError
-from readthedocs.projects.models import Project
 from readthedocs.organizations.models import Organization
+from readthedocs.projects.models import Project
 
 
 class Command(BaseCommand):

readthedocs/oauth/management/commands/sync_vcs_data.py

@@ -1,9 +1,9 @@
 import datetime
 import json
 
-from django.utils import timezone
 from django.contrib.auth.models import User
 from django.core.management.base import BaseCommand
+from django.utils import timezone
 
 from readthedocs.oauth.tasks import sync_remote_repositories
 

readthedocs/oauth/migrations/0012_create_new_table_for_remote_organization_normalization.py

@@ -1,9 +1,9 @@
 # Generated by Django 2.2.17 on 2020-12-23 10:58
 
-from django.conf import settings
-from django.db import migrations, models
 import django.db.models.deletion
 import django_extensions.db.fields
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/oauth/migrations/0013_create_new_table_for_remote_repository_normalization.py

@@ -1,9 +1,9 @@
 # Generated by Django 2.2.17 on 2020-12-21 18:16
 
-from django.conf import settings
-from django.db import migrations, models
 import django.db.models.deletion
 import django_extensions.db.fields
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/organizations/migrations/0001_squashed.py

@@ -1,9 +1,9 @@
 # Generated by Django 2.2.10 on 2020-02-25 23:59
 
 import autoslug.fields
+import django.db.models.deletion
 from django.conf import settings
 from django.db import migrations, models
-import django.db.models.deletion
 
 
 class Migration(migrations.Migration):

readthedocs/organizations/migrations/0005_historicalorganization_historicalteam.py

@@ -1,10 +1,10 @@
 # Generated by Django 2.2.24 on 2021-07-21 15:36
 
 import autoslug.fields
-from django.conf import settings
-from django.db import migrations, models
 import django.db.models.deletion
 import simple_history.models
+from django.conf import settings
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

readthedocs/organizations/tasks.py

@@ -5,7 +5,6 @@
 from readthedocs.builds.models import Build
 from readthedocs.worker import app
 
-
 log = structlog.get_logger(__name__)
 
 

readthedocs/profiles/urls/public.py

@@ -4,7 +4,6 @@
 
 from readthedocs.profiles import views
 
-
 urlpatterns = [
     re_path(
         r"^(?P<username>[+\w@.-]+)/$",

readthedocs/projects/migrations/0063_extend_domain_from_timestamp_model.py

@@ -1,7 +1,7 @@
 # Generated by Django 2.2.16 on 2020-09-10 22:37
 
-from django.db import migrations
 import django_extensions.db.fields
+from django.db import migrations
 
 
 class Migration(migrations.Migration):