plugins/semgrep: fix ambigious semgrep hook naming

csutils · Mar 7, 2024 · 2f274c4 · 2f274c4
1 parent 9409479
commit 2f274c4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/py/plugins/semgrep.py b/py/plugins/semgrep.py
@@ -88,8 +88,8 @@ def handle_args(self, parser, args, props):
             parser.error("'--semgrep-rules-repo' is required to run semgrep scan")
             return 1
 
-        # download semgrep rules
-        def fetch_semgrep_rules_hook(results, props):
+        # install semgrep cli and download semgrep rules
+        def prepare_semgrep_runtime_hook(results, props):
             # target dir where semgrep cli and its dependencies are installed
             semgrep_lib_dir = os.path.join(results.tmpdir, "semgrep_lib")
             try:
@@ -136,7 +136,7 @@ def fetch_semgrep_rules_hook(results, props):
             ]
             return 0
 
-        props.pre_mock_hooks += [fetch_semgrep_rules_hook]
+        props.pre_mock_hooks += [prepare_semgrep_runtime_hook]
 
         def scan_hook(results, mock, props):
             semgrep_lib_dir = os.path.join(results.tmpdir, "semgrep_lib")