Enable and fix many Ruff lints

Author: oprypin

properdocs/__main__.py

@@ -7,6 +7,7 @@
 import textwrap
 import traceback
 import warnings
+from typing import ClassVar
 
 import click
 
@@ -39,7 +40,7 @@ def _showwarning(message, category, filename, lineno, file=None, line=None):
         stack = [frame for frame in traceback.extract_stack() if frame.line][-4:-2]
         # Make sure the actual affected file's name is still present (the case of syntax warning):
         if not any(frame.filename == filename for frame in stack):
-            stack = stack[-1:] + [traceback.FrameSummary(filename, lineno, '')]
+            stack = [*stack[-1:], traceback.FrameSummary(filename, lineno, '')]
 
         tb = ''.join(traceback.format_list(stack))
     except Exception:
@@ -62,7 +63,7 @@ def _enable_warnings():
 
 
 class ColorFormatter(logging.Formatter):
-    colors = {
+    colors: ClassVar = {
         'CRITICAL': 'red',
         'ERROR': 'red',
         'WARNING': 'yellow',

properdocs/commands/build.py

@@ -87,9 +87,7 @@ def _build_template(
     output = template.render(context)
 
     # Run `post_template` plugin events.
-    output = config.plugins.on_post_template(output, template_name=name, config=config)
-
-    return output
+    return config.plugins.on_post_template(output, template_name=name, config=config)
 
 
 def _build_theme_template(
@@ -321,8 +319,8 @@ def build(config: ProperDocsConfig, *, serve_url: str | None = None, dirty: bool
         if excluded:
             log.info(
                 "The following pages are being built only for the preview "
-                "but will be excluded from `properdocs build` per `draft_docs` config:\n  - "
-                + "\n  - ".join(excluded)
+                "but will be excluded from `properdocs build` per `draft_docs` config:\n  - %s",
+                "\n  - ".join(excluded),
             )
 
         # Run `env` plugin events.

properdocs/commands/get_deps.py

@@ -27,7 +27,7 @@
 log = logging.getLogger(__name__)
 
 
-class YamlLoaderWithSuppressions(SafeLoader):  # type: ignore
+class YamlLoaderWithSuppressions(SafeLoader):  # type: ignore  # noqa: PGH003
     pass
 
 

properdocs/commands/gh_deploy.py

@@ -6,7 +6,7 @@
 import subprocess
 from typing import TYPE_CHECKING
 
-import ghp_import  # type: ignore
+import ghp_import  # type: ignore[import-untyped]
 from packaging import version
 
 import properdocs
@@ -43,8 +43,7 @@ def _get_current_sha(repo_path) -> str:
     )
 
     stdout, _ = proc.communicate()
-    sha = stdout.decode('utf-8').strip()
-    return sha
+    return stdout.decode('utf-8').strip()
 
 
 def _get_remote_url(remote_name: str) -> tuple[str, str] | tuple[None, None]:
@@ -78,7 +77,9 @@ def _check_version(branch: str) -> None:
 
     stdout, _ = proc.communicate()
     msg = stdout.decode('utf-8').strip()
-    m = re.search(r'\d+(\.\d+)+((a|b|rc)\d+)?(\.post\d+)?(\.dev\d+)?', msg, re.X | re.I)
+    m = re.search(
+        r'\d+(\.\d+)+((a|b|rc)\d+)?(\.post\d+)?(\.dev\d+)?', msg, re.VERBOSE | re.IGNORECASE
+    )
     previousv = version.parse(m.group()) if m else None
     currentv = version.parse(properdocs.__version__)
     if not previousv:
@@ -163,7 +164,6 @@ def gh_deploy(
         log.info('Your documentation should be available shortly.')
     else:
         username, repo = path.split('/', 1)
-        if repo.endswith('.git'):
-            repo = repo[: -len('.git')]
+        repo = repo.removesuffix('.git')
         url = f'https://{username}.github.io/{repo}/'
         log.info(f"Your documentation should shortly be available at: {url}")

properdocs/config/__init__.py

@@ -1,3 +1,3 @@
 from properdocs.config.base import Config, load_config
 
-__all__ = ['load_config', 'Config']
+__all__ = ['Config', 'load_config']

properdocs/config/base.py

@@ -136,7 +136,7 @@ def __init_subclass__(cls):
                     "All values are required, or can be wrapped into config_options.Optional"
                 )
 
-    def __new__(cls, *args, **kwargs) -> Config:
+    def __new__(cls, *args, **kwargs) -> Config:  # noqa: PYI034
         """Compatibility: allow referring to `LegacyConfig(...)` constructor as `Config(...)`."""
         if cls is Config:
             return LegacyConfig(*args, **kwargs)

properdocs/config/config_options.py

@@ -94,7 +94,7 @@ def pre_validation(self, config: Config, key_name: str):
     def run_validation(self, value: object) -> SomeConfig:
         config = self.config_class(config_file_path=self._config_file_path)
         try:
-            config.load_dict(value)  # type: ignore
+            config.load_dict(value)  # type: ignore[arg-type]
             failed, warnings = config.validate()
         except ConfigurationError as e:
             raise ValidationError(str(e))
@@ -359,7 +359,7 @@ def __init__(self, choices: Collection[T], default: T | None = None, **kwargs) -
     def run_validation(self, value: object) -> T:
         if value not in self.choices:
             raise ValidationError(f"Expected one of: {self.choices} but received: {value!r}")
-        return value  # type: ignore
+        return value  # type: ignore[return-value]
 
 
 class Deprecated(BaseConfigOption):
@@ -536,7 +536,7 @@ def run_validation(self, value: object) -> T | None:
         return self.option.validate(value)
 
     def post_validation(self, config: Config, key_name: str):
-        result = self.option.post_validation(config, key_name)  # type: ignore
+        result = self.option.post_validation(config, key_name)  # type: ignore[func-returns-value]
         self.warnings = self.option.warnings
         return result
 

properdocs/contrib/search/__init__.py

@@ -30,6 +30,7 @@ def get_lunr_supported_lang(self, lang):
             lang_part = fallback.get(lang_part, lang_part)
             if os.path.isfile(os.path.join(base_path, 'lunr-language', f'lunr.{lang_part}.js')):
                 return lang_part
+        return None
 
     def run_validation(self, value: object):
         if isinstance(value, str):
@@ -70,7 +71,7 @@ def on_config(self, config: ProperDocsConfig, **kwargs) -> ProperDocsConfig:
             path = os.path.join(base_path, 'templates')
             config.theme.dirs.append(path)
             if 'search/main.js' not in config.extra_javascript:
-                config.extra_javascript.append('search/main.js')  # type: ignore
+                config.extra_javascript.append('search/main.js')
         if self.config.lang is None:
             # lang setting undefined. Set default based on theme locale
             validate = _PluginConfig.lang.run_validation

properdocs/contrib/search/search_index.py

@@ -13,7 +13,7 @@
     from properdocs.structure.toc import AnchorLink, TableOfContents
 
 try:
-    from lunr import lunr  # type: ignore
+    from lunr import lunr  # type: ignore[import-not-found]
 
     haslunrpy = True
 except ImportError:

properdocs/localization.py

@@ -18,7 +18,7 @@
 
     has_babel = True
 except ImportError:  # pragma: no cover
-    from properdocs.utils.babel_stub import Locale, UnknownLocaleError  # type: ignore
+    from properdocs.utils.babel_stub import Locale, UnknownLocaleError  # type: ignore[assignment]
 
     has_babel = False