[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

.ruff.toml

@@ -44,5 +44,3 @@ ignore = [
 "src/stpipe/cmdline.py" = [
   "T201",
 ]
-
-

src/stpipe/_cmdline.py

@@ -498,7 +498,7 @@ def step_from_cmdline(args):
             )
 
             if known.debug:
-                import pdb  # noqa: T100
+                import pdb
 
                 pdb.post_mortem()
             else:

src/stpipe/_log.py

@@ -36,7 +36,7 @@
 # LOGS AS EXCEPTIONS
 
 
-class LoggedException(Exception):  # noqa: N818
+class LoggedException(Exception):
     """
     This is an exception used when a log record is converted into an
     exception.
@@ -85,14 +85,14 @@ def __init__(
         break_level=logging.NOTSET,
         recording_level=logging.NOTSET,
         recording_formatter=None,
-        format=None,  # noqa: A002
+        format=None,
     ):
         if isinstance(handler, str):
             handler = handler.strip().split(",")
         self.handlers = [self.get_handler(x.strip()) for x in handler]
         self.level = level
         if format is None:
-            format = DEFAULT_FORMAT  # noqa: A001
+            format = DEFAULT_FORMAT
         self.format = logging.Formatter(format)
         for handler in self.handlers:
             handler.setLevel(self.level)

src/stpipe/crds_client.py

@@ -11,14 +11,14 @@
 from crds.core.exceptions import CrdsError
 
 __all__ = [
-    "check_reference_open",
     "CrdsError",
+    "check_reference_open",
+    "get_context_used",
     "get_multiple_reference_paths",
     "get_override_name",
     "get_reference_file",
     "get_svn_version",
     "reference_uri_to_cache_path",
-    "get_context_used",
 ]
 
 

src/stpipe/exceptions.py

@@ -1,4 +1,4 @@
-class StpipeException(Exception):  # noqa: N818
+class StpipeException(Exception):
     """
     Base class for exceptions from the stpipe package.
     """

src/stpipe/format_template.py

@@ -184,7 +184,7 @@ def format(self, format_string, **kwargs):
     # Make the instance callable
     __call__ = format
 
-    def get_value(self, key, args, kwargs):  # noqa: ARG002
+    def get_value(self, key, args, kwargs):
         """Return a given field value
 
         Parameters

src/stpipe/library.py

@@ -14,11 +14,11 @@
 from .datamodel import AbstractDataModel
 
 __all__ = [
-    "LibraryError",
+    "AbstractModelLibrary",
     "BorrowError",
     "ClosedLibraryError",
+    "LibraryError",
     "NoGroupID",
-    "AbstractModelLibrary",
 ]
 
 

src/stpipe/step.py

@@ -1420,7 +1420,7 @@ def update_pars(self, parameters):
                 )
 
     @classmethod
-    def build_config(cls, input, **kwargs):  # noqa: A002
+    def build_config(cls, input, **kwargs):
         """Build the ConfigObj to initialize a Step
 
         A Step config is built in the following order:

tests/steps/__init__.py

@@ -28,7 +28,7 @@ class WithDefaultsStep(BaseStep):
     par4 = string(default='default par4 value')
     """
 
-    def process(self, input_data):  # noqa: D102
+    def process(self, input_data):
         log.info(
             "Parameters par1=%s, par2=%s, par3=%s, par4=%s",
             self.par1,
@@ -49,7 +49,7 @@ class MakeListStep(BaseStep):
     par3 = boolean(default=False) # Does it blend?
     """
 
-    def process(self, a=None, b=None):  # noqa: D102
+    def process(self, a=None, b=None):
         log.info("Arguments a=%s b=%s", a, b)
         log.info(
             "Parameters par1=%s, par2=%s, par3=%s", self.par1, self.par2, self.par3
@@ -70,7 +70,7 @@ class EmptyPipeline(BasePipeline):
     par1 = string(default='Name the atomizer') # Control the frobulization
     """
 
-    def process(self, *args):  # noqa: D102
+    def process(self, *args):
         return args
 
 
@@ -85,5 +85,5 @@ class MakeListPipeline(BasePipeline):
         "make_list": MakeListStep,
     }
 
-    def process(self, *args):  # noqa: D102
+    def process(self, *args):
         return self.make_list.run(*args)

tests/test_hooks.py

@@ -99,7 +99,7 @@ def process(self, input_data):
         result = self.shovelpixels.run(input_data)
         result = self.cancelnoise.run(result)
 
-        return result  # noqa: RET504
+        return result
 
 
 def hook_function(input_data):