Merge pull request #281 from asottile/drop-py38

drop support for python 3.8

Author: asottile

.github/workflows/main.yml

@@ -10,10 +10,10 @@ jobs:
   main-windows:
     uses: asottile/workflows/.github/workflows/[email protected]
     with:
-      env: '["py38"]'
+      env: '["py39"]'
       os: windows-latest
   main-linux:
     uses: asottile/workflows/.github/workflows/[email protected]
     with:
-      env: '["py38", "py39", "py310", "py311", "py312"]'
+      env: '["py39", "py310", "py311", "py312"]'
       os: ubuntu-latest

.pre-commit-config.yaml

@@ -17,7 +17,7 @@ repos:
     rev: v3.13.0
     hooks:
     -   id: reorder-python-imports
-        args: [--py38-plus, --add-import, 'from __future__ import annotations']
+        args: [--py39-plus, --add-import, 'from __future__ import annotations']
 -   repo: https://github.com/asottile/add-trailing-comma
     rev: v3.1.0
     hooks:
@@ -26,7 +26,7 @@ repos:
     rev: v3.16.0
     hooks:
     -   id: pyupgrade
-        args: [--py38-plus]
+        args: [--py39-plus]
 -   repo: https://github.com/hhatto/autopep8
     rev: v2.3.1
     hooks:

add_trailing_comma/_ast_helpers.py

@@ -13,13 +13,4 @@ def ast_parse(contents_text: str) -> ast.Module:
 
 
 def ast_to_offset(node: ast.AST) -> Offset:
-    candidates = [node]
-    while candidates:
-        candidate = candidates.pop()
-        if hasattr(candidate, 'lineno'):
-            return Offset(candidate.lineno, candidate.col_offset)
-        elif hasattr(candidate, '_fields'):  # pragma: <3.9 cover
-            for field in reversed(candidate._fields):
-                candidates.append(getattr(candidate, field))
-    else:
-        raise AssertionError(node)
+    return Offset(node.lineno, node.col_offset)

add_trailing_comma/_data.py

@@ -3,12 +3,10 @@
 import ast
 import collections
 import pkgutil
+from collections.abc import Iterable
 from typing import Callable
-from typing import Iterable
-from typing import List
 from typing import NamedTuple
 from typing import Protocol
-from typing import Tuple
 from typing import TypeVar
 
 from tokenize_rt import Offset
@@ -22,8 +20,8 @@ class State(NamedTuple):
 
 
 AST_T = TypeVar('AST_T', bound=ast.AST)
-TokenFunc = Callable[[int, List[Token]], None]
-ASTFunc = Callable[[State, AST_T], Iterable[Tuple[Offset, TokenFunc]]]
+TokenFunc = Callable[[int, list[Token]], None]
+ASTFunc = Callable[[State, AST_T], Iterable[tuple[Offset, TokenFunc]]]
 
 FUNCS = collections.defaultdict(list)
 

add_trailing_comma/_main.py

@@ -2,8 +2,8 @@
 
 import argparse
 import sys
-from typing import Iterable
-from typing import Sequence
+from collections.abc import Iterable
+from collections.abc import Sequence
 
 from tokenize_rt import src_to_tokens
 from tokenize_rt import Token

add_trailing_comma/_plugins/_with.py

@@ -1,8 +1,7 @@
 from __future__ import annotations
 
 import ast
-import sys
-from typing import Iterable
+from collections.abc import Iterable
 
 from tokenize_rt import Offset
 from tokenize_rt import Token
@@ -15,20 +14,20 @@
 from add_trailing_comma._token_helpers import fix_brace
 
 
-if sys.version_info >= (3, 9):  # pragma: >=3.9 cover
-    def _fix_with(i: int, tokens: list[Token]) -> None:
+def _fix_with(i: int, tokens: list[Token]) -> None:
+    i += 1
+    if tokens[i].name == 'UNIMPORTANT_WS':
         i += 1
-        if tokens[i].name == 'UNIMPORTANT_WS':
-            i += 1
-        if tokens[i].src == '(':
-            fix = find_simple(i, tokens)
-            # only fix if outer parens are for the with items (next is ':')
-            if fix is not None and tokens[fix.braces[-1] + 1].src == ':':
-                fix_brace(tokens, fix, add_comma=True, remove_comma=True)
+    if tokens[i].src == '(':
+        fix = find_simple(i, tokens)
+        # only fix if outer parens are for the with items (next is ':')
+        if fix is not None and tokens[fix.braces[-1] + 1].src == ':':
+            fix_brace(tokens, fix, add_comma=True, remove_comma=True)
 
-    @register(ast.With)
-    def visit_With(
-        state: State,
-        node: ast.With,
-    ) -> Iterable[tuple[Offset, TokenFunc]]:
-        yield ast_to_offset(node), _fix_with
+
+@register(ast.With)
+def visit_With(
+    state: State,
+    node: ast.With,
+) -> Iterable[tuple[Offset, TokenFunc]]:
+    yield ast_to_offset(node), _fix_with

add_trailing_comma/_plugins/calls.py

@@ -2,7 +2,7 @@
 
 import ast
 import functools
-from typing import Iterable
+from collections.abc import Iterable
 
 from tokenize_rt import Offset
 from tokenize_rt import Token

add_trailing_comma/_plugins/classes.py

@@ -2,7 +2,7 @@
 
 import ast
 import functools
-from typing import Iterable
+from collections.abc import Iterable
 
 from tokenize_rt import Offset
 from tokenize_rt import Token

add_trailing_comma/_plugins/functions.py

@@ -2,7 +2,7 @@
 
 import ast
 import functools
-from typing import Iterable
+from collections.abc import Iterable
 
 from tokenize_rt import Offset
 from tokenize_rt import Token

add_trailing_comma/_plugins/imports.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 import ast
-from typing import Iterable
+from collections.abc import Iterable
 
 from tokenize_rt import Offset
 from tokenize_rt import Token