RepoAuditor 0.2.12__tar.gz → 0.2.15__tar.gz

{repoauditor-0.2.12 → repoauditor-0.2.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.12
+Version: 0.2.15
 Summary: Add your description here
 Project-URL: Homepage, https://github.com/gt-sse-center/RepoAuditor
 Project-URL: Documentation, https://github.com/gt-sse-center/RepoAuditor

{repoauditor-0.2.12 → repoauditor-0.2.15}/pyproject.toml

@@ -114,7 +114,7 @@ select = [
     # "TID", # 9
     "TD",
     "TC",
-    # "ARG", # 5
+    "ARG",
     "PTH",
     "FLY",
     "I",
@@ -133,7 +133,7 @@ select = [
     # "UP", # 100
     "FURB",
     "RUF",
-    # "TRY", # 16
+    "TRY",
 
 ]
 

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/CommandLineProcessor.py

@@ -70,13 +70,13 @@ class CommandLineProcessor:
 
         for module in modules:
             if argument_separator in module.name:
-                msg = f"The module name '{module.name}' contains '{argument_separator}'."
-                raise Exception(msg)
+                msg = f"The module name '{module.name}' contains '{argument_separator}', which should be used as an argument separator."
+                raise ValueError(msg)
 
             prev_module = module_map.get(module.name, None)
             if prev_module is not None:
                 msg = f"The module '{module.name}' has already been defined."
-                raise Exception(msg)
+                raise ValueError(msg)
 
             module_map[module.name] = module
 
@@ -92,7 +92,7 @@ class CommandLineProcessor:
             this_module = module_map.get(parts[0], None)
             if this_module is None:
                 msg = f"'{parts[0]}' is not a recognized module name."
-                raise Exception(msg)
+                raise ValueError(msg)
 
             if len(parts) == 1:
                 included_modules.add(parts[0])
@@ -119,7 +119,7 @@ class CommandLineProcessor:
             this_module = module_map.get(parts[0], None)
             if this_module is None:
                 msg = f"'{parts[0]}' is not a recognized module name."
-                raise Exception(msg)
+                raise ValueError(msg)
 
             if len(parts) == 1:
                 module_map.pop(parts[0])
@@ -173,7 +173,7 @@ class CommandLineProcessor:
 
             if parts[0] not in module_map:
                 msg = f"'{parts[0]}' is not a recognized module name."
-                raise Exception(msg)
+                raise ValueError(msg)
 
             if len(parts) == 2:
                 dynamic_args.setdefault(parts[0], {})[parts[1]] = value

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/EntryPoint.py

@@ -34,7 +34,7 @@ ARGUMENT_SEPARATOR = "-"
 # ----------------------------------------------------------------------
 class NaturalOrderGrouper(TyperGroup):
     # ----------------------------------------------------------------------
-    def list_commands(self, *args, **kwargs) -> list[str]:
+    def list_commands(self, *args, **kwargs) -> list[str]:  # noqa: ARG002
         return self.commands.keys()
 
 
@@ -181,7 +181,7 @@ def _VersionCallback(value: bool) -> None:
 )
 def EntryPoint(
     ctx: typer.Context,
-    version: Annotated[
+    version: Annotated[  # noqa: ARG001
         bool,
         typer.Option(
             "--version",

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/ExecuteModules.py

@@ -130,7 +130,7 @@ def Execute(
             # ----------------------------------------------------------------------
             def Prepare(
                 context: Any,  # noqa: ANN401
-                on_simple_status_func: Callable[[str], None],
+                on_simple_status_func: Callable[[str], None],  # noqa: ARG001
             ) -> tuple[int, ExecuteTasks.TransformTasksExTypes.TransformFuncType]:
                 module_info = cast(ModuleInfo, context)
                 del context

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/Impl/ParallelSequentialProcessor.py

@@ -103,7 +103,7 @@ def _Impl(
                 ExecuteTasks.TaskData(item.name, (results_index, item))  # type: ignore[attr-defined]
                 for results_index, item in parallel
             ],
-            lambda context, status: Execute(*context),
+            lambda context, status: Execute(*context),  # noqa: ARG005
             max_num_threads=max_num_threads,
             return_exceptions=True,
         )

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/Module.py

@@ -92,7 +92,7 @@ class Module(ABC):
     def GetDynamicArgDefinitions(self) -> dict[str, TypeDefinitionItemType]:
         """Returns information about dynamic arguments that the module can consume (often from the command line)."""
 
-        raise Exception("Abstract method")  # pragma: no cover # noqa: EM101
+        raise NotImplementedError("Abstract method")  # pragma: no cover # noqa: EM101
 
     # ----------------------------------------------------------------------
     @abstractmethod
@@ -107,7 +107,7 @@ class Module(ABC):
         exception to indicate that the command line data is invalid.
         """
 
-        raise Exception("Abstract method")  # pragma: no cover # noqa: EM101
+        raise NotImplementedError("Abstract method")  # pragma: no cover # noqa: EM101
 
     # ----------------------------------------------------------------------
     def Evaluate(

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/Plugin.py

@@ -17,4 +17,4 @@ from .Module import Module
 @pluggy.HookspecMarker(APP_NAME)
 def GetModule() -> Module:
     """Returns a Module"""
-    raise Exception("hookspec")  # pragma: no cover # noqa: EM101
+    raise NotImplementedError("hookspec")  # pragma: no cover # noqa: EM101

{repoauditor-0.2.12 → repoauditor-0.2.15}/src/RepoAuditor/__init__.py

@@ -3,4 +3,4 @@ APP_NAME = "RepoAuditor"
 # Note that this value will be overwritten by calls to `python ../../Build.py update_version` based
 # on changes observed in the git repository. The default value below will be used until the value
 # here is explicitly updated by the Continuous Integration system.
-__version__ = "0.2.12"
+__version__ = "0.2.15"

{repoauditor-0.2.12 → repoauditor-0.2.15}/tests/CommandLineProcessor_UnitTest.py

@@ -369,7 +369,9 @@ def test_IgnoreAllWarnings():
 def test_ErrorInvalidModuleName():
     with pytest.raises(
         Exception,
-        match=re.escape("The module name 'Invalid-name' contains '-'."),
+        match=re.escape(
+            "The module name 'Invalid-name' contains '-', which should be used as an argument separator."
+        ),
     ):
         CommandLineProcessor.Create(
             lambda *args: {},