Lint/format all Python sources in repo (#2643)

* Update linting, formatting commands

Our linting and formatting commands were not considering new
directories like benchmarks/ which also contain python code which we
should lint and format.

* format all the things

* lint all the things

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Tahir H. Butt <[email protected]>

Author: 3 people

benchmarks/django_simple/config/settings/base.py

@@ -5,6 +5,7 @@
 
 import environ
 
+
 ROOT_DIR = Path(__file__).resolve(strict=True).parent.parent.parent
 # ddtrace_django/
 APPS_DIR = ROOT_DIR / "ddtrace_django"

benchmarks/django_simple/config/settings/production.py

@@ -1,6 +1,7 @@
 from .base import *  # noqa
 from .base import env
 
+
 # GENERAL
 # ------------------------------------------------------------------------------
 # https://docs.djangoproject.com/en/dev/ref/settings/#secret-key

benchmarks/django_simple/config/urls.py

@@ -1,10 +1,12 @@
 from django.conf import settings
 from django.conf.urls.static import static
 from django.contrib import admin
-from django.urls import include, path
+from django.urls import include
+from django.urls import path
 from django.views import defaults as default_views
 from django.views.generic import TemplateView
 
+
 urlpatterns = [
     path("", TemplateView.as_view(template_name="pages/home.html"), name="home"),
     path("about/", TemplateView.as_view(template_name="pages/about.html"), name="about"),

benchmarks/django_simple/config/wsgi.py

@@ -14,11 +14,12 @@
 
 """
 import os
-import sys
 from pathlib import Path
+import sys
 
 from django.core.wsgi import get_wsgi_application
 
+
 # This allows easy placement of apps within the interior
 # ddtrace_django directory.
 ROOT_DIR = Path(__file__).resolve(strict=True).parent.parent
@@ -40,4 +41,4 @@
 
 # Enable only profiler to be used without ddtrace-run
 if os.getenv("BENCHMARK_PROFILING_ENABLED", False) is not False:
-    import ddtrace.profiling.auto
+    import ddtrace.profiling.auto  # noqa

benchmarks/django_simple/ddtrace_django/contrib/sites/migrations/0001_initial.py

@@ -1,6 +1,7 @@
 import django.contrib.sites.models
 from django.contrib.sites.models import _simple_domain_name_validator
-from django.db import migrations, models
+from django.db import migrations
+from django.db import models
 
 
 class Migration(migrations.Migration):

benchmarks/django_simple/ddtrace_django/contrib/sites/migrations/0002_alter_domain_unique.py

@@ -1,5 +1,6 @@
 import django.contrib.sites.models
-from django.db import migrations, models
+from django.db import migrations
+from django.db import models
 
 
 class Migration(migrations.Migration):

benchmarks/django_simple/ddtrace_django/users/admin.py

@@ -1,9 +1,10 @@
+from ddtrace_django.users.forms import UserChangeForm
+from ddtrace_django.users.forms import UserCreationForm
 from django.contrib import admin
 from django.contrib.auth import admin as auth_admin
 from django.contrib.auth import get_user_model
 from django.utils.translation import gettext_lazy as _
 
-from ddtrace_django.users.forms import UserChangeForm, UserCreationForm
 
 User = get_user_model()
 

benchmarks/django_simple/ddtrace_django/users/forms.py

@@ -2,6 +2,7 @@
 from django.contrib.auth import get_user_model
 from django.utils.translation import gettext_lazy as _
 
+
 User = get_user_model()
 
 

benchmarks/django_simple/ddtrace_django/users/migrations/0001_initial.py

@@ -1,6 +1,7 @@
 import django.contrib.auth.models
 import django.contrib.auth.validators
-from django.db import migrations, models
+from django.db import migrations
+from django.db import models
 import django.utils.timezone
 
 
@@ -63,7 +64,10 @@ class Migration(migrations.Migration):
                     "is_active",
                     models.BooleanField(
                         default=True,
-                        help_text="Designates whether this user should be treated as active. Unselect this instead of deleting accounts.",
+                        help_text=(
+                            "Designates whether this user should be treated as active."
+                            " Unselect this instead of deleting accounts."
+                        ),
                         verbose_name="active",
                     ),
                 ),
@@ -79,7 +83,10 @@ class Migration(migrations.Migration):
                     "groups",
                     models.ManyToManyField(
                         blank=True,
-                        help_text="The groups this user belongs to. A user will get all permissions granted to each of their groups.",
+                        help_text=(
+                            "The groups this user belongs to."
+                            " A user will get all permissions granted to each of their groups."
+                        ),
                         related_name="user_set",
                         related_query_name="user",
                         to="auth.Group",

benchmarks/django_simple/ddtrace_django/users/urls.py

@@ -1,10 +1,8 @@
+from ddtrace_django.users.views import user_detail_view
+from ddtrace_django.users.views import user_redirect_view
+from ddtrace_django.users.views import user_update_view
 from django.urls import path
 
-from ddtrace_django.users.views import (
-    user_detail_view,
-    user_redirect_view,
-    user_update_view,
-)
 
 app_name = "users"
 urlpatterns = [