Skip to content

Commit

Permalink
Merge pull request #10 from slaren/tidy-ci
Browse files Browse the repository at this point in the history
Add clang-tidy reviews to CI
  • Loading branch information
slaren committed May 11, 2023
2 parents b608b55 + f92faf5 commit 33dd282
Show file tree
Hide file tree
Showing 3 changed files with 61 additions and 0 deletions.
18 changes: 18 additions & 0 deletions .clang-tidy
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
---
Checks: >
bugprone-*,
-bugprone-easily-swappable-parameters,
-bugprone-implicit-widening-of-multiplication-result,
-bugprone-narrowing-conversions,
readability-*,
-readability-avoid-unconditional-preprocessor-if,
-readability-function-cognitive-complexity,
-readability-identifier-length,
-readability-implicit-bool-conversion,
-readability-magic-numbers,
-readability-uppercase-literal-suffix,
clang-analyzer-*,
-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling,
performance-*,
portability-*,
FormatStyle: none
20 changes: 20 additions & 0 deletions .github/workflows/tidy-post.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: clang-tidy review post comments

on:
workflow_run:
workflows: ["clang-tidy-review"]
types:
- completed

jobs:
build:
runs-on: ubuntu-latest

steps:
- uses: ZedThree/clang-tidy-review/[email protected]
# lgtm_comment_body, max_comments, and annotations need to be set on the posting workflow in a split setup
with:
# adjust options as necessary
lgtm_comment_body: ''
annotations: false
max_comments: 25
23 changes: 23 additions & 0 deletions .github/workflows/tidy-review.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: clang-tidy-review

on:
pull_request:
branches:
- master

jobs:
clang-tidy-review:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3

- uses: ZedThree/[email protected]
id: review
with:
lgtm_comment_body: ''
build_dir: build
cmake_command: cmake . -B build -DCMAKE_EXPORT_COMPILE_COMMANDS=on
split_workflow: true

- uses: ZedThree/clang-tidy-review/[email protected]

0 comments on commit 33dd282

Please sign in to comment.