Fix usage of clang-tidy-review

Author: justsmth

.clang-tidy

@@ -11,6 +11,5 @@ Checks: '
   modernize-*,
   performance-*,
   portability-*,
-  readability-*
   '
 WarningsAsErrors: ''

.github/workflows/clang-tidy-post.yml

@@ -10,7 +10,7 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: ZedThree/clang-tidy-review/post@v0.21.0
+      - uses: justsmth/clang-tidy-review/post@e8b24ea1d3ecfaa960525cd4bd417913987ab304
         with:
           lgtm_comment_body: '' # An empty string means it won't post a LGTM message
           annotations: false

.github/workflows/clang-tidy-review.yml

@@ -9,10 +9,10 @@ jobs:
       - uses: actions/setup-go@v4
         with:
           go-version: '>=1.18'
-      - name: Build Project
-        run: ./util/build_compilation_database.sh
-      - uses: ZedThree/[email protected]
+      # Fork that avoid unnecessary filtering of clang-tidy output
+      - uses: justsmth/clang-tidy-review@e8b24ea1d3ecfaa960525cd4bd417913987ab304
         with:
           split_workflow: true
-
-      - uses: ZedThree/clang-tidy-review/[email protected]
+          clang_tidy_version: 19
+          install_commands: /github/workspace/util/build_compilation_database.sh
+      - uses: justsmth/clang-tidy-review/upload@e8b24ea1d3ecfaa960525cd4bd417913987ab304

util/build_compilation_database.sh

@@ -8,7 +8,7 @@ TMP_DIR=`mktemp -d`
 echo ${TMP_DIR}
 AWS_LC_BUILD="${TMP_DIR}/AWS-LC-BUILD"
 
-MY_CMAKE_FLAGS=("-GNinja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_EXPORT_COMPILE_COMMANDS=ON")
+MY_CMAKE_FLAGS=("-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_EXPORT_COMPILE_COMMANDS=ON" "-DDISABLE_GO=ON")
 
 mkdir -p "${AWS_LC_BUILD}"