Fix clang tidy ci (#2375)

### Issues:
Addresses P229174291

### Description of changes: 
Fixes usage of clang-tidy-review:
* Build needed to be performed inside docker image so that source files
would have correct paths in the `compile_commands.json`
Made changes to clang-tidy-review to preserve comments that are very
close to lines in the diff
*
justsmth/clang-tidy-review@e8b24ea

### Testing:
* I added a bad commit to this PR so that clang-tidy review would make a
comment. See below.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.

Author: justsmth

.clang-tidy

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

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

@@ -9,10 +9,9 @@ jobs:
       - uses: actions/setup-go@v4
         with:
           go-version: '>=1.18'
-      - name: Build Project
-        run: ./util/build_compilation_database.sh
       - uses: ZedThree/[email protected]
         with:
           split_workflow: true
-
+          clang_tidy_version: 19
+          install_commands: /github/workspace/util/build_compilation_database.sh
       - uses: ZedThree/clang-tidy-review/[email protected]

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}"