Revise check_clang_format workflow to report all errors (#86)

The clang-format checker fails after finding a file with a format error, requiring multiple commits to find all the errors. This CL updates the workflow to process all changed files in a single invocation, which should solve the problem. Signed-off-by: Derek G Foster <[email protected]>
ipdk-io · Mar 5, 2024 · 927be8e · 927be8e
1 parent 94f4460
commit 927be8e
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/.github/workflows/pipeline.yml b/.github/workflows/pipeline.yml
@@ -133,7 +133,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Get list of changed Files
-        id: changes
+        id: changed
         uses: tj-actions/changed-files@v42
         with:
           files: |
@@ -142,7 +142,6 @@ jobs:
             **.cc
 
       - name: Check for formatting errors
+        if: steps.changed.outputs.any_changed == 'true'
         run: |
-          for file in ${{ steps.changes.outputs.all_changed_files }}; do
-            clang-format --dry-run -Werror $file
-          done
+          clang-format -n -Werror ${{ steps.changed.outputs.all_changed_files }}