Add tox.

Author: JulienPalard

sphinxlint/__main__.py

@@ -54,7 +54,8 @@ def __call__(self, parser, namespace, values, option_string=None):
                     sort_fields.append(SortField[field_name.upper()])
                 except KeyError:
                     raise ValueError(
-                        f"Unsupported sort field: {field_name}, supported values are {SortField.as_supported_options()}"
+                        f"Unsupported sort field: {field_name}, "
+                        "supported values are {SortField.as_supported_options()}"
                     ) from None
             setattr(namespace, self.dest, sort_fields)
 
@@ -152,8 +153,8 @@ def _check_file(todo):
 def sort_errors(results, sorted_by):
     """Flattens and potentially sorts errors based on user prefernces"""
     if not sorted_by:
-        for results in results:
-            yield from results
+        for result in results:
+            yield from result
         return
     errors = list(error for errors in results for error in errors)
     # sorting is stable in python, so we can sort in reverse order to get the

sphinxlint/checkers.py

@@ -118,8 +118,12 @@ def check_default_role(file, lines, options=None):
             before_match = line[: match.start()]
             after_match = line[match.end() :]
             stripped_line = line.strip()
-            if (stripped_line.startswith("|") and stripped_line.endswith("|") and
-                stripped_line.count("|") >= 4 and "|" in match.group(0)):
+            if (
+                stripped_line.startswith("|")
+                and stripped_line.endswith("|")
+                and stripped_line.count("|") >= 4
+                and "|" in match.group(0)
+            ):
                 return  # we don't handle tables yet.
             if re.search(rst.ROLE_TAG + "$", before_match):
                 # It's not a default role: it starts with a tag.
@@ -267,7 +271,10 @@ def check_role_with_double_backticks(file, lines, options=None):
             before = paragraph[: inline_literal.start()]
             if re.search(rst.ROLE_TAG + "$", before):
                 error_offset = paragraph[: inline_literal.start()].count("\n")
-                yield paragraph_lno + error_offset, "role use a single backtick, double backtick found."
+                yield (
+                    paragraph_lno + error_offset,
+                    "role use a single backtick, double backtick found.",
+                )
             paragraph = (
                 paragraph[: inline_literal.start()] + paragraph[inline_literal.end() :]
             )
@@ -288,9 +295,15 @@ def check_missing_space_before_role(file, lines, options=None):
         if match:
             error_offset = paragraph[: match.start()].count("\n")
             if looks_like_glued(match):
-                yield paragraph_lno + error_offset, f"missing space before role ({match.group(0)})."
+                yield (
+                    paragraph_lno + error_offset,
+                    f"missing space before role ({match.group(0)}).",
+                )
             else:
-                yield paragraph_lno + error_offset, f"role missing opening tag colon ({match.group(0)})."
+                yield (
+                    paragraph_lno + error_offset,
+                    f"role missing opening tag colon ({match.group(0)}).",
+                )
 
 
 @checker(".rst", ".po")
@@ -459,4 +472,4 @@ def check_dangling_hyphen(file, lines, options):
     for lno, line in enumerate(lines):
         stripped_line = line.rstrip("\n")
         if re.match(r".*[a-z]-$", stripped_line):
-            yield lno + 1, f"Line ends with dangling hyphen"
+            yield lno + 1, "Line ends with dangling hyphen"

sphinxlint/sphinxlint.py

@@ -1,6 +1,8 @@
 from collections import Counter
 from dataclasses import dataclass
 from os.path import splitext
+from pathlib import Path
+from typing import Optional
 
 from sphinxlint.utils import hide_non_rst_blocks, po2rst
 
@@ -49,13 +51,12 @@ def check_text(filename, text, checkers, options=None):
     return errors
 
 
-def check_file(filename, checkers, options: CheckersOptions = None):
+def check_file(filename, checkers, options: Optional[CheckersOptions] = None):
     ext = splitext(filename)[1]
     if not any(ext in checker.suffixes for checker in checkers):
         return Counter()
     try:
-        with open(filename, encoding="utf-8") as f:
-            text = f.read()
+        text = Path(filename).read_text(encoding="UTF-8")
         if filename.endswith(".po"):
             text = po2rst(text)
     except OSError as err:

tests/test_enable_disable.py

@@ -1,5 +1,5 @@
-from random import choice
 import re
+from random import choice
 
 from sphinxlint.__main__ import main
 

tests/test_filter_out_literal.py

@@ -1,6 +1,5 @@
 from sphinxlint.utils import hide_non_rst_blocks
 
-
 LITERAL = r"""
 Hide non-RST Blocks
 ===================

tests/test_sphinxlint.py

@@ -1,10 +1,9 @@
 from pathlib import Path
 
-from sphinxlint.utils import paragraphs
-
 import pytest
 
 from sphinxlint.__main__ import main
+from sphinxlint.utils import paragraphs
 
 FIXTURE_DIR = Path(__file__).resolve().parent / "fixtures"
 
@@ -69,8 +68,8 @@ def test_sphinxlint_shall_not_pass(file, expected_errors, capsys):
 
 @pytest.mark.parametrize("file", [str(FIXTURE_DIR / "paragraphs.rst")])
 def test_paragraphs(file):
-    with open(file) as f:
-        lines = f.readlines()
+    with open(file, encoding="UTF-8") as ifile:
+        lines = ifile.readlines()
     actual = paragraphs(lines)
     for lno, para in actual:
         firstpline = para.splitlines(keepends=True)[0]

tests/test_xpass_friends.py

@@ -3,14 +3,13 @@
 This is useful to avoid a sphinx-lint release to break many CIs.
 """
 
-from pathlib import Path
 import shlex
+from pathlib import Path
 
 import pytest
 
 from sphinxlint.__main__ import main
 
-
 FIXTURE_DIR = Path(__file__).resolve().parent / "fixtures"
 
 

tox.ini

@@ -0,0 +1,26 @@
+[tox]
+envlist = py3{8,9,10,11,12}, mypy, black, pylint
+isolated_build = True
+skip_missing_interpreters = True
+
+[testenv]
+deps = pytest
+commands = pytest
+
+[testenv:black]
+deps = black
+skip_install = True
+commands = black --check --diff sphinxlint/ tests/
+
+[testenv:mypy]
+deps =
+  mypy
+  types-polib
+skip_install = True
+commands = mypy --exclude fixtures --ignore-missing-imports sphinxlint/ tests/
+
+[testenv:pylint]
+deps =
+  pylint
+  pytest
+commands = pylint --ignore fixtures sphinxlint/ tests/ --disable missing-function-docstring,missing-module-docstring,missing-class-docstring,too-few-public-methods,too-many-return-statements --good-names=po