refactor: Save exception messages as variables

* https://docs.astral.sh/ruff/rules/raw-string-in-exception/
* https://docs.astral.sh/ruff/rules/f-string-in-exception/

shell_logger/html_utilities.py

@@ -114,7 +114,8 @@ def _append_html(
             elif isinstance(arg, Iterable):
                 _append_html(f, *arg)
             else:
-                raise RuntimeError(f"Unsupported type: {type(arg)}")
+                message = f"Unsupported type: {type(arg)}"
+                raise RuntimeError(message)
 
     with open(output, "a") as output_file:
         _append_html(output_file, *args)

shell_logger/shell.py

@@ -204,11 +204,12 @@ def run(self, command: str, **kwargs) -> SimpleNamespace:
         # KeyboardInterrupt.
         except KeyboardInterrupt:
             os.close(self.aux_stdin_wfd)
-            raise RuntimeError(
+            message = (
                 f"There was a problem running the command `{command}`.  "
                 "This is a fatal error and we cannot continue.  Ensure that "
                 "the syntax of the command is correct."
-            ) from None
+            )
+            raise RuntimeError(message) from None
         finish = round(time() * milliseconds_per_second)
 
         # Pull the return code and return the results.  Note that if the

shell_logger/shell_logger.py

@@ -120,9 +120,8 @@ def append(path: Path) -> ShellLogger:
             try:
                 path = next(path.glob("*.html"))
             except StopIteration as error:
-                raise RuntimeError(
-                    f"{path} does not have an html file."
-                ) from error
+                message = f"{path} does not have an html file."
+                raise RuntimeError(message) from error
         if path.is_symlink():
             path = path.resolve(strict=True)
         if path.is_file() and path.name[-5:] == ".html":
@@ -290,10 +289,11 @@ def change_log_dir(self, new_log_dir: Path) -> None:
                 :class:`ShellLogger`.
         """
         if not self.is_parent():
-            raise RuntimeError(
+            message = (
                 "You should not change the log directory of a child "
                 "`ShellLogger`; only that of the parent."
             )
+            raise RuntimeError(message)
 
         # This only gets executed once by the top-level parent
         # `ShellLogger` object.

shell_logger/trace.py

@@ -35,9 +35,11 @@ def trace_collector(**kwargs) -> Trace:
         collector = collectors[0]
         return collector(**kwargs)
     elif len(collectors) == 0:
-        raise RuntimeError(f"Unsupported trace type:  {trace_name}")
+        message = f"Unsupported trace type:  {trace_name}"
+        raise RuntimeError(message)
     else:
-        raise RuntimeError(f"Multiple trace types match '{trace_name}'.")
+        message = f"Multiple trace types match '{trace_name}'."
+        raise RuntimeError(message)
 
 
 class Trace: