From 5d1ca28978c5e649e5f9dd29c5eee670512c021c Mon Sep 17 00:00:00 2001 From: Michael Bradley Date: Sun, 23 Feb 2025 13:23:20 -0500 Subject: [PATCH] Add linting via clang-tidy --- .forgejo/workflows/build.yaml | 19 +++++++++++++++++-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/.forgejo/workflows/build.yaml b/.forgejo/workflows/build.yaml index 821439b..6de6cda 100644 --- a/.forgejo/workflows/build.yaml +++ b/.forgejo/workflows/build.yaml @@ -15,7 +15,22 @@ jobs: sudo apt-get update sudo apt-get -y install clang-format - name: Check the code's formatting - run: "clang-format --dry-run src/*.h src/*.cpp 2>&1 | tee fmt.log; [ ! -s fmt.log ]" + run: clang-format --dry-run src/*.h src/*.cpp 2>&1 | tee fmt.log; [ ! -s fmt.log ] + + lint: + name: Lint + runs-on: ubuntu-22.04 + steps: + - name: Check out the code + uses: actions/checkout@v4 + - name: Install clang-tidy and the toolchain + run: | + sudo apt-get update + sudo apt-get -y install clang-tidy cmake freeglut3-dev libglew-dev libglm-dev + - name: Set up the build environment + run: cmake -S . -B cmake-build-debug -DCMAKE_EXPORT_COMPILE_COMMANDS=ON + - name: Have clang-tidy lint for errors + run: clang-tidy -p cmake-build-cli/ src/*.cpp src/*.h --warnings-as-errors='*' build: name: Build @@ -28,7 +43,7 @@ jobs: sudo apt-get update sudo apt-get -y install cmake freeglut3-dev libglew-dev libglm-dev - name: Set up the build environment - run: cmake -S . -B cmake-build-debug -DCMAKE_EXPORT_COMPILE_COMMANDS=ON + run: cmake -S . -B cmake-build-debug - name: Compile the code run: cmake --build cmake-build-debug - name: Save executable