fix: TEST unfix valid commit ref

GitGuardian · Jan 8, 2024 · b05d44d · b05d44d
1 parent 5b6e6b7
commit b05d44d
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/ggshield/core/git_hooks/ci/previous_commit.py b/ggshield/core/git_hooks/ci/previous_commit.py
@@ -121,14 +121,9 @@ def gitlab_previous_commit_sha(verbose: bool) -> Optional[str]:
         # Targeted branch is empty
         if merge_req_base_sha == EMPTY_SHA:
             return None
-        if is_valid_git_commit_ref(merge_req_base_sha):
-            return merge_req_base_sha
-
-    if (
-        push_before_sha
-        and push_before_sha != EMPTY_SHA
-        and is_valid_git_commit_ref(push_before_sha)
-    ):
+        return merge_req_base_sha
+
+    if push_before_sha and push_before_sha != EMPTY_SHA:
         return push_before_sha
 
     # push_before_sha is also always EMPTY_SHA for the first commit of a new branch