chore: fix black and flake8 issues for the new versions of the tools (#362)

Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>

Author: behnazh-w

docs/source/conf.py

@@ -57,6 +57,7 @@
 }
 html_static_path = ["_static"]
 
+
 # We add the docstrings for class constructors in the `__init__` methods.
 def skip(app, what, name, obj, would_skip, options):
     if name == "__init__":

src/macaron/parsers/bashparser.py

@@ -1,4 +1,4 @@
-# Copyright (c) 2022 - 2022, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2022 - 2023, Oracle and/or its affiliates. All rights reserved.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/.
 
 """This module is a Python wrapper for the compiled bashparser binary.
@@ -157,7 +157,6 @@ def extract_bash_from_ci(
     # Parse the bash script files called from the current script.
     if recursive and repo_path:
         for cmd in caller_commands:
-
             # Parse the scripts that end with `.sh`.
             # We only parse recursively at depth 1, so don't set the recursive argument in parse_file().
             # TODO: parse Makefiles for bash commands.

src/macaron/slsa_analyzer/checks/build_as_code_check.py

@@ -70,7 +70,6 @@ def _has_deploy_command(self, commands: list[list[str]], build_tool: BaseBuildTo
         # Account for Python projects having separate tools for packaging and publishing.
         deploy_tool = build_tool.publisher if build_tool.publisher else build_tool.builder
         for com in commands:
-
             # Check for empty or invalid commands.
             if not com or not com[0]:
                 continue

src/macaron/slsa_analyzer/checks/provenance_l3_check.py

@@ -351,7 +351,6 @@ class Feedback(NamedTuple):
                 with tempfile.TemporaryDirectory() as temp_path:
                     downloaded_provs = []
                     for prov_asset in prov_assets:
-
                         # Check the size before downloading.
                         if self._size_large(prov_asset["size"]):
                             logger.info("Skip verifying the provenance %s: asset size too large.", prov_asset["name"])

src/macaron/slsa_analyzer/database_store.py

@@ -50,7 +50,6 @@ def store_analyze_context_to_db(db_man: DatabaseManager, analysis: AnalysisTable
 
     # Store check result table
     for check in analyze_ctx.check_results.values():
-
         check_table = CheckResultTable()
         check_table.check_id = check["check_id"]
         check_table.repository = analyze_ctx.repository_table.id

tests/slsa_analyzer/checks/base_check/test_base_check.py

@@ -1,4 +1,4 @@
-# Copyright (c) 2022 - 2022, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2022 - 2023, Oracle and/or its affiliates. All rights reserved.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl/.
 
 """This module contains the tests for BaseCheck."""
@@ -12,8 +12,10 @@
 class TestConfiguration(TestCase):
     """This class contains the tests for BaseCheck."""
 
+    # Disable flake8's D202 check: "No blank lines allowed after function docstring"
     def test_raise_implementation_error(self) -> None:
-        """Test raising errors if child class does not override abstract method(s)."""
+        """Test raising errors if child class does not override abstract method(s)."""  # noqa: D202
+
         # pylint: disable=abstract-method
         class ChildCheck(BaseCheck):
             """This class is a child class that does not implement abstract methods in Base Check."""