chore: Update Ruff linters

Author: jmgate

.pre-commit-config.yaml

@@ -9,7 +9,7 @@ repos:
     - repo: https://github.com/astral-sh/ruff-pre-commit
       rev: v0.12.2
       hooks:
-        - id: ruff
+        - id: ruff-check
         - id: ruff-format
 
     - repo: https://github.com/gitleaks/gitleaks

pyproject.toml

@@ -67,32 +67,62 @@ line-length = 79
 [tool.ruff.lint]
 extend-select = [
     "A",
+    "AIR",
+    # "ANN",
+    "ARG",
+    "ASYNC",
     "B",
     "BLE",
     "C4",
     "C90",
     "D",
+    "DJ",
     # "DTZ",
     "E",
     "EM",
     "ERA",
     "EXE",
+    "F",
+    # "FA",
     "FBT",
+    "FIX",
+    "FLY",
+    "FURB",
+    "G",
+    "I",
+    "ICN",
+    "INP",
+    "INT",
+    "ISC",
+    "LOG",
+    # "N",
     "NPY",
+    "PD",
+    # "PERF",
     "PGH",
+    "PIE",
     "PL",
     "PT",
     "PTH",
+    "PYI",
+    "Q",
     "RET",
     "RSE",
     "RUF",
     # "S",
     "SIM",
+    "SLF",
+    "SLOT",
+    "T10",
+    # "T20",
+    "TC",
+    "TD",
     "TID",
     "TCH",
     "TRY",
     # "UP",
     "W",
+    "YTT",
 ]
 ignore = [
     "D212",
@@ -106,14 +136,22 @@ extend-allowed-calls = [
 
 
 [tool.ruff.lint.per-file-ignores]
-"**/test_*.py" = ["S101"]
+"**/test_*.py" = [
+    "S101",
+    "SLF001",
+]
+"doc/source/conf.py" = ["INP001"]
 "html_utilities.py" = ["PLR2004"]
 
 
 [tool.ruff.lint.pydocstyle]
 convention = "google"
 
 
+[tool.ruff.lint.pyupgrade]
+keep-runtime-typing = true
+
+
 
 [tool.semantic_release]
 build_command = "python3 -m pip install poetry && poetry build"

shell_logger/shell.py

@@ -20,7 +20,6 @@
 from types import SimpleNamespace
 from typing import IO, Optional, TextIO
 
-
 END_OF_READ = 4
 
 

shell_logger/shell_logger.py

@@ -248,7 +248,7 @@ def update_done_time(self) -> None:
         """
         self.done_time = datetime.now()
 
-    def __update_duration(self) -> None:
+    def update_duration(self) -> None:
         """
         Update the :attr:`duration` attribute.
 
@@ -427,7 +427,7 @@ def to_html(self) -> Union[Iterator[str], list[Iterator[str]]]:
             if isinstance(log, ShellLogger):
                 # Update the duration of this ShellLogger's commands.
                 if log.duration is None:
-                    log.__update_duration()
+                    log.update_duration()
                 html.append(child_logger_card(log))
 
             # Otherwise, if this is a message being logged...
@@ -737,7 +737,7 @@ def default(self, obj: object) -> object:  # noqa: PLR0911
         """
         if isinstance(obj, ShellLogger):
             return {
-                **{"__type__": "ShellLogger"},
+                "__type__": "ShellLogger",
                 **{k: self.default(v) for k, v in obj.__dict__.items()},
             }
         if isinstance(obj, (int, float, str, bytes)):

shell_logger/stats_collector.py

@@ -317,7 +317,6 @@ def __init__(self, interval: float, manager: SyncManager) -> None:
 
         def collect(self) -> None:
             """Don't collect any disk statistics."""
-            pass
 
         def unproxied_stats(self) -> None:
             """
@@ -352,7 +351,6 @@ def __init__(self, interval: float, manager: SyncManager) -> None:
 
         def collect(self) -> None:
             """Don't collect any CPU statistics."""
-            pass
 
         def unproxied_stats(self) -> None:
             """
@@ -387,7 +385,6 @@ def __init__(self, interval: float, manager: SyncManager) -> None:
 
         def collect(self) -> None:
             """Don't collect any memory statistics."""
-            pass
 
         def unproxied_stats(self) -> None:
             """

test/__init__.py

@@ -0,0 +1,13 @@
+"""
+Create the ``test`` package.
+
+This ``__init__.py`` file creates the ``test`` package, such that tests
+can relative-import from modules in the sibling ``shell_logger``
+directory.
+"""
+
+# © 2023 National Technology & Engineering Solutions of Sandia, LLC
+# (NTESS).  Under the terms of Contract DE-NA0003525 with NTESS, the
+# U.S. Government retains certain rights in this software.
+
+# SPDX-License-Identifier: BSD-3-Clause