reorder pre-commit config

Committed via https://github.com/asottile/all-repos

Author: asottile

.pre-commit-config.yaml

@@ -6,18 +6,33 @@ repos:
     -   id: end-of-file-fixer
     -   id: check-yaml
     -   id: debug-statements
--   repo: https://github.com/PyCQA/flake8
-    rev: 4.0.1
+    -   id: double-quote-string-fixer
+    -   id: name-tests-test
+    -   id: requirements-txt-fixer
+-   repo: https://github.com/asottile/setup-cfg-fmt
+    rev: v1.20.0
     hooks:
-    -   id: flake8
-        exclude: ^docs/conf.py
--   repo: https://github.com/asottile/pyupgrade
-    rev: v2.31.1
+    -   id: setup-cfg-fmt
+-   repo: https://github.com/asottile/reorder_python_imports
+    rev: v3.0.1
     hooks:
-    -   id: pyupgrade
-        args: [--py36-plus]
+    -   id: reorder-python-imports
+        args: [--py37-plus, --add-import, 'from __future__ import annotations']
 -   repo: https://github.com/asottile/add-trailing-comma
     rev: v2.2.1
     hooks:
     -   id: add-trailing-comma
         args: [--py36-plus]
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v2.31.1
+    hooks:
+    -   id: pyupgrade
+        args: [--py37-plus]
+-   repo: https://github.com/pre-commit/mirrors-autopep8
+    rev: v1.6.0
+    hooks:
+    -   id: autopep8
+-   repo: https://github.com/PyCQA/flake8
+    rev: 4.0.1
+    hooks:
+    -   id: flake8

bin/build-manylinux-wheels

@@ -5,6 +5,8 @@ Run me after putting the source distribution on pypi.
 
 See: https://www.python.org/dev/peps/pep-0513/
 """
+from __future__ import annotations
+
 import os
 import pipes
 import subprocess

bin/download-windows-wheels

@@ -1,4 +1,6 @@
 #!/usr/bin/env python3
+from __future__ import annotations
+
 import argparse
 import io
 import json

docs/conf.py

@@ -9,6 +9,9 @@
 #
 # All configuration values have a default; values that are commented out
 # serve to show the default.
+from __future__ import annotations
+import sass
+
 import os
 import sys
 import warnings
@@ -18,7 +21,6 @@
 # documentation root, use os.path.abspath to make it absolute, like shown here.
 sys.path.insert(0, os.path.abspath('..'))
 
-import sass
 
 # -- General configuration -----------------------------------------------------
 
@@ -179,23 +181,23 @@
 # -- Options for LaTeX output --------------------------------------------------
 
 latex_elements = {
-# The paper size ('letterpaper' or 'a4paper').
-#'papersize': 'letterpaper',
+    # The paper size ('letterpaper' or 'a4paper').
+    # 'papersize': 'letterpaper',
 
-# The font size ('10pt', '11pt' or '12pt').
-#'pointsize': '10pt',
+    # The font size ('10pt', '11pt' or '12pt').
+    # 'pointsize': '10pt',
 
-# Additional stuff for the LaTeX preamble.
-#'preamble': '',
+    # Additional stuff for the LaTeX preamble.
+    # 'preamble': '',
 }
 
 # Grouping the document tree into LaTeX files. List of tuples
 # (source start file, target name, title, author, documentclass [howto/manual]).
 latex_documents = [
-  (
-      'index', 'libsass.tex', 'libsass Documentation',
-      'Hong Minhee', 'manual',
-  ),
+    (
+        'index', 'libsass.tex', 'libsass Documentation',
+        'Hong Minhee', 'manual',
+    ),
 ]
 
 # The name of an image file (relative to this directory) to place at the top of
@@ -240,11 +242,11 @@
 # (source start file, target name, title, author,
 #  dir menu entry, description, category)
 texinfo_documents = [
-  (
-      'index', 'libsass', 'libsass Documentation',
-      'Hong Minhee', 'libsass', 'One line description of project.',
-      'Miscellaneous',
-  ),
+    (
+        'index', 'libsass', 'libsass Documentation',
+        'Hong Minhee', 'libsass', 'One line description of project.',
+        'Miscellaneous',
+    ),
 ]
 
 # Documents to append as an appendix to all manuals.

pysassc.py

@@ -88,6 +88,7 @@
 .. _SassC: https://github.com/sass/sassc
 
 """
+from __future__ import annotations
 
 import functools
 import optparse

sass.py

@@ -10,6 +10,7 @@
 'a b {\n  color: blue; }\n'
 
 """
+from __future__ import annotations
 
 import collections.abc
 import inspect
@@ -56,6 +57,7 @@ class CompileError(ValueError):
     """The exception type that is raised by :func:`compile()`.
     It is a subtype of :exc:`exceptions.ValueError`.
     """
+
     def __init__(self, msg):
         super().__init__(to_native_s(msg))
 
@@ -573,7 +575,7 @@ def my_importer(path, prev):
         elif source_comments in ('line_numbers', 'default'):
             deprecation_message = (
                 'you can simply pass True to '
-                "source_comments instead of " +
+                'source_comments instead of ' +
                 repr(source_comments)
             )
             source_comments = True
@@ -789,7 +791,7 @@ def __new__(cls, r, g, b, a):
 
 class SassList(
     collections.namedtuple(
-            'SassList', ('items', 'separator', 'bracketed'),
+        'SassList', ('items', 'separator', 'bracketed'),
     ),
 ):
 

sasstests.py

@@ -1,11 +1,12 @@
+from __future__ import annotations
+
 import base64
 import collections.abc
 import contextlib
 import functools
 import glob
 import io
 import json
-import os
 import os.path
 import re
 import shutil
@@ -21,7 +22,8 @@
 
 import pysassc
 import sass
-from sassutils.builder import Manifest, build_directory
+from sassutils.builder import build_directory
+from sassutils.builder import Manifest
 from sassutils.wsgi import SassMiddleware
 
 
@@ -424,11 +426,11 @@ def importer_with_srcmap(path):
                     path,
                     'a { color: red; }',
                     json.dumps({
-                        "version": 3,
-                        "sources": [
-                            path + ".db",
+                        'version': 3,
+                        'sources': [
+                            path + '.db',
                         ],
-                        "mappings": ";AAAA,CAAC,CAAC;EAAE,KAAK,EAAE,GAAI,GAAI",
+                        'mappings': ';AAAA,CAAC,CAAC;EAAE,KAAK,EAAE,GAAI,GAAI',
                     }),
                 ),
             )
@@ -448,13 +450,13 @@ def importer(path):
 
         with assert_raises_compile_error(
             RegexMatcher(
-                    r'^Error: \n'
-                    r'       Traceback \(most recent call last\):\n'
-                    r'.+'
-                    r'ValueError: Bad path: hi\n'
-                    r'        on line 1:9 of stdin\n'
-                    r'>> @import "hi";\n'
-                    r'   --------\^\n',
+                r'^Error: \n'
+                r'       Traceback \(most recent call last\):\n'
+                r'.+'
+                r'ValueError: Bad path: hi\n'
+                r'        on line 1:9 of stdin\n'
+                r'>> @import "hi";\n'
+                r'   --------\^\n',
             ),
         ):
             sass.compile(string='@import "hi";', importers=((0, importer),))
@@ -465,14 +467,14 @@ def importer(path):
 
         with assert_raises_compile_error(
             RegexMatcher(
-                    r'^Error: \n'
-                    r'       Traceback \(most recent call last\):\n'
-                    r'.+'
-                    r'ValueError: Expected importer result to be a tuple of '
-                    r'length \(1, 2, 3\) but got 0: \(\)\n'
-                    r'        on line 1:9 of stdin\n'
-                    r'>> @import "hi";\n'
-                    r'   --------\^\n',
+                r'^Error: \n'
+                r'       Traceback \(most recent call last\):\n'
+                r'.+'
+                r'ValueError: Expected importer result to be a tuple of '
+                r'length \(1, 2, 3\) but got 0: \(\)\n'
+                r'        on line 1:9 of stdin\n'
+                r'>> @import "hi";\n'
+                r'   --------\^\n',
             ),
         ):
             sass.compile(string='@import "hi";', importers=((0, importer),))
@@ -483,14 +485,14 @@ def importer(path):
 
         with assert_raises_compile_error(
             RegexMatcher(
-                    r'^Error: \n'
-                    r'       Traceback \(most recent call last\):\n'
-                    r'.+'
-                    r'ValueError: Expected importer result to be a tuple of '
-                    r"length \(1, 2, 3\) but got 4: \('a', 'b', 'c', 'd'\)\n"
-                    r'        on line 1:9 of stdin\n'
-                    r'>> @import "hi";\n'
-                    r'   --------\^\n',
+                r'^Error: \n'
+                r'       Traceback \(most recent call last\):\n'
+                r'.+'
+                r'ValueError: Expected importer result to be a tuple of '
+                r"length \(1, 2, 3\) but got 4: \('a', 'b', 'c', 'd'\)\n"
+                r'        on line 1:9 of stdin\n'
+                r'>> @import "hi";\n'
+                r'   --------\^\n',
             ),
         ):
             sass.compile(string='@import "hi";', importers=((0, importer),))
@@ -1116,7 +1118,7 @@ def test_error(self):
 class SassFunctionTest(unittest.TestCase):
 
     def test_from_lambda(self):
-        lambda_ = lambda abc, d: None  # pragma: no branch  # noqa: E731
+        def lambda_(abc, d): return None  # pragma: no branch  # noqa: E731
         sf = sass.SassFunction.from_lambda('func_name', lambda_)
         assert 'func_name' == sf.name
         assert ('$abc', '$d') == sf.arguments
@@ -1378,14 +1380,14 @@ class CustomFunctionsTest(unittest.TestCase):
     def test_raises(self):
         with assert_raises_compile_error(
             RegexMatcher(
-                    r'^Error: error in C function raises: \n'
-                    r'       Traceback \(most recent call last\):\n'
-                    r'.+'
-                    r'AssertionError: foo\n'
-                    r'        on line 1:14 of stdin, in function `raises`\n'
-                    r'        from line 1:14 of stdin\n'
-                    r'>> a { content: raises\(\); }\n'
-                    r'   -------------\^\n$',
+                r'^Error: error in C function raises: \n'
+                r'       Traceback \(most recent call last\):\n'
+                r'.+'
+                r'AssertionError: foo\n'
+                r'        on line 1:14 of stdin, in function `raises`\n'
+                r'        from line 1:14 of stdin\n'
+                r'>> a { content: raises\(\); }\n'
+                r'   -------------\^\n$',
             ),
         ):
             compile_with_func('a { content: raises(); }')

sassutils/__init__.py

@@ -5,3 +5,4 @@
 which depends on libsass core (:mod:`sass` module).
 
 """
+from __future__ import annotations

sassutils/builder.py

@@ -2,6 +2,7 @@
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 """
+from __future__ import annotations
 
 import collections.abc
 import os.path

sassutils/distutils.py

@@ -67,18 +67,19 @@
    Added ``--output-style``/``-s`` option to :class:`build_sass` command.
 
 """
+from __future__ import annotations
 
-import distutils.errors
-import distutils.log
-import distutils.util
 import functools
 import os.path
 
+import distutils.errors
+import distutils.log
+import distutils.util
 from setuptools import Command
 from setuptools.command.sdist import sdist
 
-from sass import OUTPUT_STYLES
 from .builder import Manifest
+from sass import OUTPUT_STYLES
 
 __all__ = 'build_sass', 'validate_manifests'