Skip to content

Commit

Permalink
fix: let clang tidy reach headers
Browse files Browse the repository at this point in the history
  • Loading branch information
luishfonseca committed Sep 27, 2023
1 parent a05b5d0 commit 96ada62
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions .github/workflows/clang-tidy-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,18 +29,17 @@ jobs:
CC: gcc-11
CXX: g++-11

- name: Create results directory
run: |
mkdir clang-tidy-result
- name: Analyze
run: |
git diff -U0 HEAD^ | clang-tidy-diff -p1 -path build -export-fixes clang-tidy-result/fixes.yml
run-clang-tidy -j$(nproc) -p=build \
-header-filter="(core|engine)/include/.*" \
files "($(git diff -U0 --name-only HEAD^ | paste -s -d\|))" \
-export-fixes clang-tidy-fixes.yaml
- name: Run clang-tidy-pr-comments action
uses: platisd/clang-tidy-pr-comments@master
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
clang_tidy_fixes: clang-tidy-result/fixes.yml
clang_tidy_fixes: clang-tidy-fixes.yaml
request_changes: true
suggestions_per_comment: 10

0 comments on commit 96ada62

Please sign in to comment.