From 2b87bbd2ce434479dfd0719b6aefe90a991e5c6f Mon Sep 17 00:00:00 2001
From: Mourad Gouicem <mourad.gouicem@intel.com>
Date: Tue, 8 Oct 2024 14:07:11 +0200
Subject: [PATCH] github: apply commit msg check to all commits

---
 ...{pr-title-check.py => commit-msg-check.py} | 21 +++++++++++++------
 .../{pr-check.yml => commit-msg-check.yml}    | 11 ++++++++--
 2 files changed, 24 insertions(+), 8 deletions(-)
 rename .github/automation/{pr-title-check.py => commit-msg-check.py} (80%)
 rename .github/workflows/{pr-check.yml => commit-msg-check.yml} (71%)

diff --git a/.github/automation/pr-title-check.py b/.github/automation/commit-msg-check.py
similarity index 80%
rename from .github/automation/pr-title-check.py
rename to .github/automation/commit-msg-check.py
index 8b3786d86ce..bb98dd274b9 100755
--- a/.github/automation/pr-title-check.py
+++ b/.github/automation/commit-msg-check.py
@@ -2,6 +2,7 @@
 
 # *******************************************************************************
 # Copyright 2024 Arm Limited and affiliates.
+# Copyright 2024 Intel Corporation
 # SPDX-License-Identifier: Apache-2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -18,7 +19,7 @@
 # *******************************************************************************
 
 import argparse
-
+import subprocess
 
 # * Ensuring the scopes end in colon and same level scopes are comma delimited.
 # TODO: Limit scopes to an acceptable list of tags.
@@ -61,12 +62,20 @@ def __numCharacterCheck(msg: str):
 
 def main():
     parser = argparse.ArgumentParser()
-    parser.add_argument("msg", help="Commit message to check.")
+    parser.add_argument("head", help="Head commit of PR branch")
+    parser.add_argument("base", help="Base commit of PR branch")
     args = parser.parse_args()
-    msg: str = args.msg
-    print(f"msg: {msg}")
-    __numCharacterCheck(msg)
-    __scopeCheck(msg)
+    base: str = args.base
+    head: str = args.head
+
+    commit_range = base + ".." + head
+    messages = subprocess.run(["git", "rev-list", "--ancestry-path", commit_range, "--format=oneline"], capture_output=True, text=True).stdout
+
+    for i in messages.splitlines():
+      commit_msg=i.split(' ', 1)[1]
+      print(f"msg: {commit_msg}")
+      __numCharacterCheck(commit_msg)
+      __scopeCheck(commit_msg)
 
 
 if __name__ == "__main__":
diff --git a/.github/workflows/pr-check.yml b/.github/workflows/commit-msg-check.yml
similarity index 71%
rename from .github/workflows/pr-check.yml
rename to .github/workflows/commit-msg-check.yml
index 1a0f43e8c2f..59a659d913a 100644
--- a/.github/workflows/pr-check.yml
+++ b/.github/workflows/commit-msg-check.yml
@@ -1,5 +1,6 @@
 # *******************************************************************************
 # Copyright 2024 Arm Limited and affiliates.
+# Copyright 2024 Intel Corporation
 # SPDX-License-Identifier: Apache-2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -24,10 +25,16 @@ on:
 # Declare default permissions as read only.
 permissions: read-all
 
+# Kill stale checks
+concurrency:
+  group: ${{ github.ref }}
+  cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
+
 jobs:
   title:
+    if: github.repository == 'oneapi-src/oneDNN'
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - name: Pass pull request title through script.
-        run: python3 ./.github/automation/pr-title-check.py "${{ github.event.pull_request.title }}"
+      - name: Pass pull request commit messages through script.
+        run: python3 ./.github/automation/commit-msg-check.py "${{ github.event.pull_request.head.sha }}" "${{ github.event.pull_request.base.sha }}"