RepoAuditor 0.2.7__tar.gz → 0.2.9__tar.gz

{repoauditor-0.2.7 → repoauditor-0.2.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.7
+Version: 0.2.9
 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.7 → repoauditor-0.2.9}/pyproject.toml

@@ -109,11 +109,11 @@ select = [
     "RSE",
     "RET",
     "SLF",
-    # "SIM", # 4
+    "SIM",
     "SLOT",
     # "TID", # 9
     "TD",
-    # "TC", # 2
+    "TC",
     # "ARG", # 5
     "PTH",
     "FLY",
@@ -131,7 +131,7 @@ select = [
     # "PGH", # 5
     # "PL", # 11
     # "UP", # 100
-    # "FURB", # 1
+    "FURB",
     "RUF",
     # "TRY", # 16
 
@@ -149,6 +149,7 @@ ignore = [
     "N999", # Invalid module name
     "RSE102", # Unnecessary parentheses on raise exception
     "S101", # Use of assert detected
+    "TC006", # Add quotes to type expression in `typing.cast()`
     "UP032", # Use f-string instead of `format` call
 ]
 

{repoauditor-0.2.7 → repoauditor-0.2.9}/src/RepoAuditor/EntryPoint.py

@@ -293,11 +293,7 @@ def EntryPoint(
                 display_rationale=not no_rationale,
             )
         except Exception as ex:
-            if dm.is_debug:
-                error = traceback.format_exc()
-            else:
-                error = str(ex)
-
+            error = traceback.format_exc() if dm.is_debug else str(ex)
             dm.WriteError(error)
 
 

{repoauditor-0.2.7 → repoauditor-0.2.9}/src/RepoAuditor/ExecuteModules.py

@@ -179,69 +179,71 @@ def Execute(
                 all_results[all_results_index] = transformed_results
 
         for index, (all_results_index, module_info) in enumerate(sequential):
-            with modules_dm.Nested(
-                "Processing '{}' ({} of {})...".format(
-                    module_info.module.name,
-                    index + 1 + len(parallel),
-                    len(module_infos),
-                ),
-            ) as this_module_dm:
+            with (
+                modules_dm.Nested(
+                    "Processing '{}' ({} of {})...".format(
+                        module_info.module.name,
+                        index + 1 + len(parallel),
+                        len(module_infos),
+                    ),
+                ) as this_module_dm,
                 # rich.progress needs to output to sys.stdout
-                with this_module_dm.YieldStdout() as stdout_context:
-                    stdout_context.persist_content = False
-
-                    # Technically speaking, it would be more correct to use `stdout_context.stream` here
-                    # rather than referencing `sys.stdout` directly, but it is really hard to work with mocked
-                    # stream as mocks will create mocks for everything called on the mock. Use sys.stdout
-                    # directly to avoid that particular problem.
-                    from unittest.mock import Mock, MagicMock
-
-                    assert stdout_context.stream is sys.stdout or isinstance(
-                        stdout_context.stream, (Mock, MagicMock)
-                    ), stdout_context.stream
-
-                    with Progress(
-                        *Progress.get_default_columns(),
-                        TimeElapsedColumn(),
-                        "{task.fields[status]}",
-                        console=Capabilities.Get(sys.stdout).CreateRichConsole(sys.stdout),
-                        transient=True,
-                        refresh_per_second=10,
-                    ) as progress_bar:
-                        progress_bar_task_id = progress_bar.add_task(
-                            stdout_context.line_prefix,
-                            status="",
-                            total=module_info.module.GetNumRequirements(),
-                            visible=True,
+                this_module_dm.YieldStdout() as stdout_context,
+            ):
+                stdout_context.persist_content = False
+
+                # Technically speaking, it would be more correct to use `stdout_context.stream` here
+                # rather than referencing `sys.stdout` directly, but it is really hard to work with mocked
+                # stream as mocks will create mocks for everything called on the mock. Use sys.stdout
+                # directly to avoid that particular problem.
+                from unittest.mock import Mock, MagicMock
+
+                assert stdout_context.stream is sys.stdout or isinstance(
+                    stdout_context.stream, (Mock, MagicMock)
+                ), stdout_context.stream
+
+                with Progress(
+                    *Progress.get_default_columns(),
+                    TimeElapsedColumn(),
+                    "{task.fields[status]}",
+                    console=Capabilities.Get(sys.stdout).CreateRichConsole(sys.stdout),
+                    transient=True,
+                    refresh_per_second=10,
+                ) as progress_bar:
+                    progress_bar_task_id = progress_bar.add_task(
+                        stdout_context.line_prefix,
+                        status="",
+                        total=module_info.module.GetNumRequirements(),
+                        visible=True,
+                    )
+
+                    # ----------------------------------------------------------------------
+                    def OnStatus(
+                        num_completed: int,
+                        num_success: int,
+                        num_error: int,
+                        num_warning: int,
+                        num_does_not_apply: int,
+                    ) -> None:
+                        progress_bar.update(
+                            progress_bar_task_id,
+                            completed=num_completed,
+                            status=_CreateStatusString(
+                                num_success,
+                                num_error,
+                                num_warning,
+                                num_does_not_apply,
+                            ),
                         )
 
-                        # ----------------------------------------------------------------------
-                        def OnStatus(
-                            num_completed: int,
-                            num_success: int,
-                            num_error: int,
-                            num_warning: int,
-                            num_does_not_apply: int,
-                        ) -> None:
-                            progress_bar.update(
-                                progress_bar_task_id,
-                                completed=num_completed,
-                                status=_CreateStatusString(
-                                    num_success,
-                                    num_error,
-                                    num_warning,
-                                    num_does_not_apply,
-                                ),
-                            )
-
-                        # ----------------------------------------------------------------------
-
-                        evaluate_results = Evaluate(module_info, OnStatus)
-
-                        assert all_results[all_results_index] is None
-                        all_results[all_results_index] = evaluate_results
-
-                        this_module_dm.result = CalcResultInfo(evaluate_results)[0]
+                    # ----------------------------------------------------------------------
+
+                    evaluate_results = Evaluate(module_info, OnStatus)
+
+                    assert all_results[all_results_index] is None
+                    all_results[all_results_index] = evaluate_results
+
+                    this_module_dm.result = CalcResultInfo(evaluate_results)[0]
 
     final_results: list[list[Module.EvaluateInfo]] = []
 

{repoauditor-0.2.7 → repoauditor-0.2.9}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionQuery.py

@@ -117,7 +117,7 @@ class ClassicBranchProtectionQuery(Query):
         self,
         module_data: dict[str, Any],
     ) -> Optional[dict[str, Any]]:
-        branch = module_data.get("branch", None)
+        branch = module_data.get("branch")
         if branch is None:
             # Get the default branch name
             response = module_data["session"].get("")

{repoauditor-0.2.7 → repoauditor-0.2.9}/src/RepoAuditor/Plugins/GitHub/Module.py

@@ -69,7 +69,7 @@ class GitHubModule(Module):
     # ----------------------------------------------------------------------
     @override
     def GenerateInitialData(self, dynamic_args: dict[str, Any]) -> Optional[dict[str, Any]]:
-        dynamic_args["session"] = _GitHubSession(dynamic_args["url"], dynamic_args.get("pat", None))
+        dynamic_args["session"] = _GitHubSession(dynamic_args["url"], dynamic_args.get("pat"))
 
         return dynamic_args
 
@@ -108,8 +108,7 @@ class _GitHubSession(requests.Session):
 
             self.headers["Authorization"] = f"Bearer {github_pat}"
 
-        if github_url.endswith("/"):
-            github_url = github_url[:-1]
+        github_url = github_url.removesuffix("/")
 
         url_parts = urlparse(github_url)
         path_parts = url_parts.path.split("/")

{repoauditor-0.2.7 → repoauditor-0.2.9}/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.7"
+__version__ = "0.2.9"