chatterino2
Code Issues

Clean up clang-tidy action checks (#4331)

Browse Source 
* clang-tidy action: Exclude lib & tests dir for clang-tidy CI checks

The lib dir is obvious, but the tests dir is disabled because gtest has
a million different thing that clang-tidy doesn't like, while they're in
reality perfectly reasonable

* clang-tidy-action: Disable the LGTM comment
This commit is contained in:
pajlada
2023-01-26 19:47:32 +01:00
committed by GitHub
parent bf5a5b839c
commit adf58d2770
2 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/build.yml
+1
View File
@@ -166,6 +166,7 @@ jobs:
build_dir: build build_dir: build
config_file: ".clang-tidy" config_file: ".clang-tidy"
split_workflow: true split_workflow: true
exclude: "tests/*,lib/*"
- uses: actions/upload-artifact@v3 - uses: actions/upload-artifact@v3
if: (startsWith(matrix.os, 'ubuntu') && matrix.pch == false && matrix.qt-version == '5.15.2' && github.event_name == 'pull_request') if: (startsWith(matrix.os, 'ubuntu') && matrix.pch == false && matrix.qt-version == '5.15.2' && github.event_name == 'pull_request')
.github/workflows/post-clang-tidy-review.yml
+2
View File
@@ -37,3 +37,5 @@ jobs:
- uses: ZedThree/clang-tidy-review/post@v0.10.1 - uses: ZedThree/clang-tidy-review/post@v0.10.1
id: review id: review
with:
lgtm_comment_body: ""