Add pyupgrade and django-upgrade pre-commit hooks (#1553)

Author: matthiask

.pre-commit-config.yaml

@@ -14,6 +14,16 @@ repos:
     rev: 0.10.1
     hooks:
     -   id: doc8
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v2.29.1
+    hooks:
+    -   id: pyupgrade
+        args: [--py36-plus]
+-   repo: https://github.com/adamchainz/django-upgrade
+    rev: 1.4.0
+    hooks:
+    -   id: django-upgrade
+        args: [--target-version, "3.2"]
 -   repo: https://github.com/pycqa/isort
     rev: 5.10.1
     hooks:

debug_toolbar/management/commands/debugsqlshell.py

@@ -27,7 +27,7 @@ def execute(self, sql, params=()):
             end_time = time()
             duration = (end_time - start_time) * 1000
             formatted_sql = sqlparse.format(raw_sql, reindent=True)
-            print("{} [{:.2f}ms]".format(formatted_sql, duration))
+            print(f"{formatted_sql} [{duration:.2f}ms]")
 
 
 base_module.CursorDebugWrapper = PrintQueryWrapper

debug_toolbar/panels/cache.py

@@ -63,7 +63,7 @@ def __init__(self, cache):
         self.cache = cache
 
     def __repr__(self):
-        return str("<CacheStatTracker for %s>") % repr(self.cache)
+        return "<CacheStatTracker for %s>" % repr(self.cache)
 
     def _get_func_info(self):
         frame = sys._getframe(3)

debug_toolbar/panels/profiling.py

@@ -30,7 +30,7 @@ def parent_classes(self):
 
     def background(self):
         r, g, b = hsv_to_rgb(*self.hsv)
-        return "rgb({:f}%,{:f}%,{:f}%)".format(r * 100, g * 100, b * 100)
+        return f"rgb({r * 100:f}%,{g * 100:f}%,{b * 100:f}%)"
 
     def func_std_string(self):  # match what old profile produced
         func_name = self.func

debug_toolbar/panels/signals.py

@@ -97,7 +97,7 @@ def generate_stats(self, request, response):
                     receiver_class_name = getattr(
                         receiver.__self__, "__class__", type
                     ).__name__
-                    text = "{}.{}".format(receiver_class_name, receiver_name)
+                    text = f"{receiver_class_name}.{receiver_name}"
                 else:
                     text = receiver_name
                 receivers.append(text)

debug_toolbar/panels/sql/utils.py

@@ -26,8 +26,8 @@ def reformat_sql(sql, with_toggle=False):
     if not with_toggle:
         return formatted
     simple = simplify(parse_sql(sql, aligned_indent=False))
-    uncollapsed = '<span class="djDebugUncollapsed">{}</span>'.format(simple)
-    collapsed = '<span class="djDebugCollapsed djdt-hidden">{}</span>'.format(formatted)
+    uncollapsed = f'<span class="djDebugUncollapsed">{simple}</span>'
+    collapsed = f'<span class="djDebugCollapsed djdt-hidden">{formatted}</span>'
     return collapsed + uncollapsed
 
 

debug_toolbar/panels/sql/views.py

@@ -49,11 +49,11 @@ def sql_explain(request, verified_data):
                 # SQLite's EXPLAIN dumps the low-level opcodes generated for a query;
                 # EXPLAIN QUERY PLAN dumps a more human-readable summary
                 # See https://www.sqlite.org/lang_explain.html for details
-                cursor.execute("EXPLAIN QUERY PLAN {}".format(sql), params)
+                cursor.execute(f"EXPLAIN QUERY PLAN {sql}", params)
             elif vendor == "postgresql":
-                cursor.execute("EXPLAIN ANALYZE {}".format(sql), params)
+                cursor.execute(f"EXPLAIN ANALYZE {sql}", params)
             else:
-                cursor.execute("EXPLAIN {}".format(sql), params)
+                cursor.execute(f"EXPLAIN {sql}", params)
             headers = [d[0] for d in cursor.description]
             result = cursor.fetchall()
 

debug_toolbar/panels/templates/panel.py

@@ -42,7 +42,7 @@ def _request_context_bind_template(self, template):
     self.context_processors = OrderedDict()
     updates = {}
     for processor in processors:
-        name = "{}.{}".format(processor.__module__, processor.__name__)
+        name = f"{processor.__module__}.{processor.__name__}"
         context = processor(self.request)
         self.context_processors[name] = context
         updates.update(context)

debug_toolbar/panels/templates/views.py

@@ -44,7 +44,7 @@ def template_source(request):
         except TemplateDoesNotExist:
             pass
     else:
-        source = "Template Does Not Exist: {}".format(template_origin_name)
+        source = f"Template Does Not Exist: {template_origin_name}"
 
     try:
         from pygments import highlight

debug_toolbar/utils.py

@@ -27,7 +27,7 @@ def get_module_path(module_name):
     try:
         module = import_module(module_name)
     except ImportError as e:
-        raise ImproperlyConfigured("Error importing HIDE_IN_STACKTRACES: {}".format(e))
+        raise ImproperlyConfigured(f"Error importing HIDE_IN_STACKTRACES: {e}")
     else:
         source_path = inspect.getsourcefile(module)
         if source_path.endswith("__init__.py"):
@@ -157,7 +157,7 @@ def get_name_from_obj(obj):
 
     if hasattr(obj, "__module__"):
         module = obj.__module__
-        name = "{}.{}".format(module, name)
+        name = f"{module}.{name}"
 
     return name