Use dry-run to check for clang-format errors instead of inplace editing

Laupetin · Oct 2, 2023 · 8ec83ba · 8ec83ba
1 parent 55d0a83
commit 8ec83ba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.github/workflows/ensure-formatting.yaml b/.github/workflows/ensure-formatting.yaml
@@ -15,5 +15,4 @@ jobs:
       - name: Test formatting for all files
         working-directory: ${{ github.workspace }}
         run: |
-          ./scripts/reformat-all.sh
-          [[ $(git status --porcelain | wc -l) == 0 ]]
+          ./scripts/check-format.sh
diff --git a/scripts/check-format.sh b/scripts/check-format.sh
@@ -0,0 +1,6 @@
+#!/bin/bash
+
+# Go to repository root
+cd "$(dirname "$0")/.." || exit 2
+
+find ./src ./test -iname '*.h' -o -iname '*.cpp' | xargs clang-format -Werror -ferror-limit=1 --dry-run