Skip to content

Commit

Permalink
ci: avoid clang-tidy-diff; use xargs -n run-clang-tidy instead (Jeffe…
Browse files Browse the repository at this point in the history
…rsonLab#21)

Changed the github workflow to use "xargs -n run-clang-tidy" instead of "clang-tidy-diff".
  • Loading branch information
wdconinc authored Jul 17, 2024
1 parent 47a2e31 commit 4c8ff65
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ jobs:
with:
release-platform: ${{ env.clang-tidy-release }}/${{ env.clang-tidy-platform }}
run: |
git diff ${{ github.event.pull_request.head.sha }} ${{ github.event.pull_request.base.sha }} | clang-tidy-diff -p 1 -path build -quiet -export-fixes clang_tidy_fixes.yml -extra-arg=-std=c++20 -extra-arg=--gcc-toolchain=${COMPILER_PATH} -checks='-*,bugprone-*,-bugprone-narrowing-conversions,-bugprone-macro-parentheses' -clang-tidy-binary run-clang-tidy
git diff --name-only ${{ github.event.pull_request.head.sha }} ${{ github.event.pull_request.base.sha }} | xargs -n 1000 run-clang-tidy -p build -export-fixes clang_tidy_fixes.yml -extra-arg=-std=c++20 -extra-arg=--gcc-toolchain=${COMPILER_PATH} -checks='-*,bugprone-*,-bugprone-narrowing-conversions,-bugprone-macro-parentheses'
- name: Run clang-tidy on all files
uses: aidasoft/run-lcg-view@v3
if: ${{ github.event_name == 'push' }}
Expand Down

0 comments on commit 4c8ff65

Please sign in to comment.