Skip to content

Commit 996a432

Browse files
committed
Fix cicd - clang-tidy should run after build success
1 parent 84b58c3 commit 996a432

1 file changed

Lines changed: 20 additions & 20 deletions

File tree

.github/workflows/cpp-build-test-coverage.yml

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -57,26 +57,7 @@ jobs:
5757
--inline-suppr \
5858
--error-exitcode=1 \
5959
./src ./include
60-
61-
# -------------------------------------------------------
62-
# Step 2.1: Run Clang-Tidy (experimenting , not exit)
63-
# - Modern C++ static analysis
64-
# - Enforces best practices
65-
# - Add `--warnings-as-errors=*` to exit code 1
66-
# -------------------------------------------------------
67-
- name: Run clang-tidy
68-
# run: |
69-
# echo "Running clang-tidy..."
70-
# clang-tidy \
71-
# -checks='clang-analyzer-*,modernize-*,performance-*,readability-*' \
72-
# -p build \
73-
# $(find ./src -name '*.cpp')
74-
run: |
75-
echo "Running clang-tidy using .clang-tidy options"
76-
clang-tidy \
77-
-p build \
78-
-header-filter='^src/.*' $(find src -name "*.cpp")
79-
60+
8061
# -------------------------------------------------------
8162
# Step 3: Configure and build the project
8263
# -------------------------------------------------------
@@ -116,3 +97,22 @@ jobs:
11697
cd build
11798
echo "## Test Coverage Summary" >> $GITHUB_STEP_SUMMARY
11899
lcov --summary coverageFiltered.info >> $GITHUB_STEP_SUMMARY
100+
101+
# -------------------------------------------------------
102+
# Step 7: Run Clang-Tidy (experimenting , not exit)
103+
# - Modern C++ static analysis
104+
# - Enforces best practices
105+
# - Add `--warnings-as-errors=*` to exit code 1
106+
# -------------------------------------------------------
107+
- name: Run clang-tidy
108+
# run: |
109+
# echo "Running clang-tidy..."
110+
# clang-tidy \
111+
# -checks='clang-analyzer-*,modernize-*,performance-*,readability-*' \
112+
# -p build \
113+
# $(find ./src -name '*.cpp')
114+
run: |
115+
echo "Running clang-tidy using .clang-tidy options"
116+
clang-tidy \
117+
-p build \
118+
-header-filter='^src/.*' $(find src -name "*.cpp")

0 commit comments

Comments
 (0)