From 6b31660c591961e41f29775e7df99bf974486648 Mon Sep 17 00:00:00 2001
From: MyungJoo Ham <myungjoo.ham@samsung.com>
Date: Mon, 15 Jan 2024 21:05:07 +0900
Subject: [PATCH] github/actions/static: clean up clang-format checker

This is to be squahed before push

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
---
 .github/workflows/static.check.yml | 36 +++++++++++++++++-------------
 1 file changed, 20 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/static.check.yml b/.github/workflows/static.check.yml
index c6702d92c3..538ad0553d 100644
--- a/.github/workflows/static.check.yml
+++ b/.github/workflows/static.check.yml
@@ -10,33 +10,37 @@ jobs:
     runs-on: ubuntu-latest
     name: Static checks
     steps:
-      - uses: actions/checkout@v4
+      - name: Preparing step 1...
+        uses: actions/checkout@v4
         with:
-          fetch-depth: 5
-      - name: Get changed files
-        id: changed-files
-        uses: tj-actions/changed-files@v41
-        with:
-          since_last_remote_commit: true
-      - name: List all files changed
+          fetch-depth: 0
+      - name: Preparing step 2... Saving to files
+        shell: bash
         env:
-          ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_files }}
+          BEFORE_REF: ${{ github.event.before }}
+          AFTER_REF: ${{ github.event.after }}
+          REFS: ${{ github.event.pull_request.commits }}
         run: |
-          for file in "$ALL_CHANGED_FILES"; do
+          if [ -z $REF ]; then REF=1; fi
+          git show --pretty="format:" --name-only --diff-filter=AMRC -${REF} | sort | uniq | grep '.' > changed.files
+          for file in `cat changed.files`; do
             echo "$file was changed"
           done
-      - name: Run clang-format for cc/hh/hpp/cpp files
-        env:
-          ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_changed_files }}
+
+      - name: /Checker/ clang-format for .cc/.hh/.hpp/.cpp files
         shell: bash
+        env:
+          BEFORE_REF: ${{ github.event.before }}
+          AFTER_REF: ${{ github.event.after }}
+          REFS: ${{ github.event.pull_request.commits }}
         run: |
           sudo apt update && sudo apt-get install clang-format
           echo "Check .clang-format file"
           if [ ! -f ".clang-format" ]; then
-            echo ".clang-format file not found"
+            echo "::error .clang-format file not found"
             exit 1
           fi
-          for file in "$ALL_CHANGED_FILES"; do
+          for file in `cat changed.files`; do
             if [[ "$file" =~ .*\.hh$ ]] || [[ "$file" =~ .*\.hpp ]] || [[ "$file" =~ .*\.cc$ ]] || [[ "$file" =~ .*\.cpp ]]; then
               echo "$file appears to be a C++ file. Applying clang-format"
               clang-format -i ${file}
@@ -47,7 +51,7 @@ jobs:
           git diff -- *.cc *.hh *.hpp *.cpp > .ci.clang-format.patch
           SIZE=$(stat -c%s .ci.clang-format.patch)
           if [[ $SIZE -ne 0 ]]; then
-            echo "clang-format shows that the commit has style errors."
+            echo "::error clang-format has found style errors in C++ files."
             cat .ci.clang-format.patch
             exit 1
           fi