Exclude pybind in clang-tidy

Author: haochengxia

.github/workflows/code-quality.yml

@@ -32,6 +32,18 @@ jobs:
             **/*.cc
             **/*.hpp
 
+      - name: Get changed C/C++ files excluding python binding
+        id: changed-files-without-pybind
+        uses: tj-actions/changed-files@v46.0.5
+        with:
+          files: |
+            **/*.c
+            **/*.h
+            **/*.cpp
+            **/*.cc
+            **/*.hpp
+            !libCacheSim-python/**
+
       - name: Create log directory
         run: mkdir -p logs
 
@@ -67,12 +79,12 @@ jobs:
             -DCMAKE_EXPORT_COMPILE_COMMANDS=ON
 
       - name: Clang-tidy check
-        if: steps.changed-files.outputs.any_changed == 'true'
+        if: steps.changed-files-without-pybind.outputs.any_changed == 'true'
         run: |
           echo "Running clang-tidy checks..."
           FILES_WITH_ISSUES=0
           
-          for file in ${{ steps.changed-files.outputs.all_changed_files }}; do
+          for file in ${{ steps.changed-files-without-pybind.outputs.all_changed_files }}; do
             echo "Checking $file with clang-tidy"
             LOG_FILE="logs/tidy_$(basename "$file").log"
             

libCacheSim-python/CMakeLists.txt

@@ -187,6 +187,7 @@ target_link_libraries(_libcachesim PRIVATE
     cachesim_c
     cachesim_cpp
     pybind11::headers
+    pybind11::module
     ${GLIB2_LIBRARIES}
     ${XGBOOST_LIBRARIES}
     ${LIGHTGBM_LIBRARIES}