pass code through ruff

Author: dhellmann

integration_tests/build_django.py

@@ -1,7 +1,6 @@
 #!/usr/bin/env python3
 #
-"""Try to build the Django documentation.
-"""
+"""Try to build the Django documentation."""
 
 import argparse
 import os
@@ -10,70 +9,83 @@
 import tempfile
 
 
-def doit(*cmd, description='', cwd=None):
-    print(f'\n[{description}]\nrunning: {" ".join(cmd)}')
+def doit(*cmd, description="", cwd=None):
+    print(f"\n[{description}]\nrunning: {' '.join(cmd)}")
     completed = subprocess.run(cmd, cwd=cwd)
     try:
         completed.check_returncode()
     except subprocess.CalledProcessError as err:
-        raise RuntimeError(f'command failed {description}') from err
+        raise RuntimeError(f"command failed {description}") from err
 
 
 def try_build(workdir, srcdir, django_repo):
-    print(f'working in {workdir}')
+    print(f"working in {workdir}")
     doit(
-        'git', 'clone', '--depth', '1', django_repo, 'django',
-        description='clone django',
+        "git",
+        "clone",
+        "--depth",
+        "1",
+        django_repo,
+        "django",
+        description="clone django",
         cwd=workdir,
     )
-    djangodir = workdir + '/django'
+    djangodir = workdir + "/django"
     doit(
-        'tox', '-e', 'docs', '--notest',
-        description='build django docs virtualenv',
+        "tox",
+        "-e",
+        "docs",
+        "--notest",
+        description="build django docs virtualenv",
         cwd=djangodir,
     )
     doit(
-        '.tox/docs/bin/pip', 'uninstall', '-y', 'sphinxcontrib-spelling',
-        description='uninstall packaged sphinxcontrib-spelling',
+        ".tox/docs/bin/pip",
+        "uninstall",
+        "-y",
+        "sphinxcontrib-spelling",
+        description="uninstall packaged sphinxcontrib-spelling",
         cwd=djangodir,
     )
     doit(
-        '.tox/docs/bin/pip', 'install', srcdir,
-        description='install sphinxcontrib-spelling from source',
+        ".tox/docs/bin/pip",
+        "install",
+        srcdir,
+        description="install sphinxcontrib-spelling from source",
         cwd=djangodir,
     )
     doit(
-        'tox', '-e', 'docs',
-        description='build django docs',
+        "tox",
+        "-e",
+        "docs",
+        description="build django docs",
         cwd=djangodir,
     )
 
 
 def main(args=sys.argv[1:]):
     parser = argparse.ArgumentParser()
-    parser.add_argument('--debug', action='store_true', default=False,
-                        help='show full tracebacks')
-    parser.add_argument('--src', help='source directory')
-    parser.add_argument('--django-repo',
-                        default='https://github.com/django/django.git')
+    parser.add_argument(
+        "--debug", action="store_true", default=False, help="show full tracebacks"
+    )
+    parser.add_argument("--src", help="source directory")
+    parser.add_argument("--django-repo", default="https://github.com/django/django.git")
     parsed = parser.parse_args(args)
 
     srcdir = parsed.src
     if not srcdir:
-        srcdir = os.path.realpath(
-            os.path.dirname(os.path.dirname(sys.argv[0]))
-        )
+        srcdir = os.path.realpath(os.path.dirname(os.path.dirname(sys.argv[0])))
 
     try:
         with tempfile.TemporaryDirectory() as dirname:
             try_build(dirname, srcdir, parsed.django_repo)
     except Exception as err:
         if parsed.debug:
             raise
-        print(f'ERROR: {err}')
+        print(f"ERROR: {err}")
         return 1
     return 0
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     sys.exit(main())

pyproject.toml

@@ -76,7 +76,11 @@ dependencies = [
 ]
 [tool.hatch.envs.test.scripts]
 test = "python -m pytest --cov=sphinxcontrib.spelling --cov-report term-missing --log-level DEBUG tests"
-lint = "ruff check sphinxcontrib integration_tests tests"
+lint = [
+    "ruff check sphinxcontrib integration_tests tests",
+    "ruff format --check sphinxcontrib integration_tests tests",
+]
+lint-fix = ["ruff format sphinxcontrib integration_tests tests"]
 pkglint = [
     "hatch build",
     "twine check dist/*.tar.gz dist/*.whl",
@@ -87,3 +91,6 @@ pkglint = [
 dependencies = ["tox"]
 [tool.hatch.envs.integration.scripts]
 django = "./integration_tests/build_django.py"
+
+[tool.ruff]
+exclude = ["sphinxcontrib/spelling/version.py"]

sphinxcontrib/spelling/__init__.py

@@ -13,51 +13,51 @@
 
 
 def setup(app):
-    version = importlib_metadata.version('sphinxcontrib-spelling')
-    logger.info('Initializing Spelling Checker %s', version)
+    version = importlib_metadata.version("sphinxcontrib-spelling")
+    logger.info("Initializing Spelling Checker %s", version)
     app.add_builder(builder.SpellingBuilder)
     # Register the 'spelling' directive for setting parameters within
     # a document
-    app.add_directive('spelling', directive.LegacySpellingDirective)
+    app.add_directive("spelling", directive.LegacySpellingDirective)
     app.add_domain(domain.SpellingDomain)
     # Register an environment collector to merge data gathered by the
     # directive in parallel builds
     app.add_env_collector(asset.SpellingCollector)
     # Report guesses about correct spelling
-    app.add_config_value('spelling_show_suggestions', False, 'env')
+    app.add_config_value("spelling_show_suggestions", False, "env")
     # Limit the number of suggestions output
-    app.add_config_value('spelling_suggestion_limit', 0, 'env')
+    app.add_config_value("spelling_suggestion_limit", 0, "env")
     # Report the whole line that has the error
-    app.add_config_value('spelling_show_whole_line', True, 'env')
+    app.add_config_value("spelling_show_whole_line", True, "env")
     # Emit misspelling as a sphinx warning instead of info message
-    app.add_config_value('spelling_warning', False, 'env')
+    app.add_config_value("spelling_warning", False, "env")
     # Set the language for the text
-    app.add_config_value('spelling_lang', 'en_US', 'env')
+    app.add_config_value("spelling_lang", "en_US", "env")
     # Set the language for the tokenizer
-    app.add_config_value('tokenizer_lang', 'en_US', 'env')
+    app.add_config_value("tokenizer_lang", "en_US", "env")
     # Set a user-provided list of words known to be spelled properly
-    app.add_config_value('spelling_word_list_filename', None, 'env')
+    app.add_config_value("spelling_word_list_filename", None, "env")
     # Assume anything that looks like a PyPI package name is spelled properly
-    app.add_config_value('spelling_ignore_pypi_package_names', False, 'env')
+    app.add_config_value("spelling_ignore_pypi_package_names", False, "env")
     # Assume words that look like wiki page names are spelled properly
-    app.add_config_value('spelling_ignore_wiki_words', True, 'env')
+    app.add_config_value("spelling_ignore_wiki_words", True, "env")
     # Assume words that are all caps, or all caps with trailing s, are
     # spelled properly
-    app.add_config_value('spelling_ignore_acronyms', True, 'env')
+    app.add_config_value("spelling_ignore_acronyms", True, "env")
     # Assume words that are part of __builtins__ are spelled properly
-    app.add_config_value('spelling_ignore_python_builtins', True, 'env')
+    app.add_config_value("spelling_ignore_python_builtins", True, "env")
     # Assume words that look like the names of importable modules are
     # spelled properly
-    app.add_config_value('spelling_ignore_importable_modules', True, 'env')
+    app.add_config_value("spelling_ignore_importable_modules", True, "env")
     # Treat contributor names from git history as spelled correctly
-    app.add_config_value('spelling_ignore_contributor_names', True, 'env')
+    app.add_config_value("spelling_ignore_contributor_names", True, "env")
     # Add any user-defined filter classes
-    app.add_config_value('spelling_filters', [], 'env')
+    app.add_config_value("spelling_filters", [], "env")
     # Set a user-provided list of files to ignore
-    app.add_config_value('spelling_exclude_patterns', [], 'env')
+    app.add_config_value("spelling_exclude_patterns", [], "env")
     # Choose whether or not the misspelled output should be displayed
     # in the terminal
-    app.add_config_value('spelling_verbose', True, 'env')
+    app.add_config_value("spelling_verbose", True, "env")
     return {
         "parallel_read_safe": True,
         "parallel_write_safe": True,

sphinxcontrib/spelling/asset.py

@@ -13,7 +13,6 @@
 
 
 class SpellingCollector(EnvironmentCollector):
-
     def clear_doc(self, app, env, docname) -> None:
         with contextlib.suppress(AttributeError, KeyError):
             del env.spelling_document_words[docname]
@@ -24,7 +23,7 @@ def merge_other(self, app, env, docnames, other):
         except AttributeError:
             other_words = {}
 
-        if not hasattr(env, 'spelling_document_words'):
+        if not hasattr(env, "spelling_document_words"):
             env.spelling_document_words = collections.defaultdict(list)
         env.spelling_document_words.update(other_words)