Merge pull request #2755 from zjeffer/master
Added clang-tidy GitHub action
This commit is contained in:
commit
2674982c7b
|
@ -1,4 +1,4 @@
|
|||
name: Linter
|
||||
name: clang-format
|
||||
|
||||
on: [push, pull_request]
|
||||
|
||||
|
@ -8,7 +8,8 @@ jobs:
|
|||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
- uses: DoozyX/clang-format-lint-action@v0.16.2
|
||||
name: clang-format
|
||||
with:
|
||||
source: '.'
|
||||
extensions: 'hpp,h,cpp,c'
|
||||
clangFormatVersion: 16
|
||||
clangFormatVersion: 16
|
|
@ -0,0 +1,31 @@
|
|||
name: clang-tidy
|
||||
|
||||
on: [push, pull_request]
|
||||
|
||||
jobs:
|
||||
build:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: alexays/waybar:debian
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
- name: configure
|
||||
run: |
|
||||
meson -Dcpp_std=c++20 build # necessary to generate compile_commands.json
|
||||
ninja -C build # necessary to find certain .h files (xdg, wayland, etc.)
|
||||
- uses: zjeffer/cpp-linter-action@test/process-compilation-database
|
||||
name: clang-tidy
|
||||
id: clang-tidy-check
|
||||
env:
|
||||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||
PIP_NO_CACHE_DIR: false
|
||||
with:
|
||||
style: "" # empty string => don't do clang-format checks here, we do them in clang-format.yml
|
||||
files-changed-only: true # only check files that have changed
|
||||
lines-changed-only: true # only check lines that have changed
|
||||
tidy-checks: "" # empty string => use the .clang-tidy file
|
||||
version: "17" # clang-tools version
|
||||
database: "build" # path to the compile_commands.json file
|
||||
- name: Check if clang-tidy failed on any files
|
||||
if: steps.clang-tidy-check.outputs.checks-failed > 0
|
||||
run: echo "Some files failed the linting checks!" && exit 1
|
Loading…
Reference in New Issue