Merge branch 'main' into Development

Reviewed-on: #6
Co-authored-by: Romain BOULLARD <romain.boullard@protonmail.com>
Co-committed-by: Romain BOULLARD <romain.boullard@protonmail.com>

.clang-tidy

@@ -15,6 +15,7 @@ Checks: >
   -cppcoreguidelines-macro-usage,
   -cppcoreguidelines-avoid-do-while
 
+HeaderFilterRegex: '^(Bigfoot)/'
 ExcludeHeaderFilterRegex: '_generated.*'
 
 CheckOptions:

.gitea/workflows/sonarqube.yml

@@ -3,8 +3,8 @@ name: Bigfoot
 on:
   push:
     branches:
-      - main
+      - '**'
-      - Development
+  workflow_dispatch:
 
 jobs:
   build-and-test:
@@ -25,23 +25,14 @@ jobs:
 
       - name: Generate
         run: |
-            conan install . --deployer=full_deploy --deployer-folder=build --remote=bigfootpackages -pr:h=clang_coverage -pr:b=clang_coverage  --build=missing -s build_type=Debug -o bigfoot/*:build_tests=True -o bigfoot/*:tracy=False -o bigfoot/*:build_tools=True -o bigfoot/*:vulkan=True -o bigfoot/*:build_benchmarks=True
+            conan install . --deployer=full_deploy --deployer-folder=build --remote=bigfootpackages -pr:h=clang -pr:b=clang  --build=missing -s build_type=Debug -o bigfoot/*:build_tests=True -o bigfoot/*:tracy=False -o bigfoot/*:build_tools=True -o bigfoot/*:vulkan=True -o bigfoot/*:build_benchmarks=True
-            cmake -S . -B ./build/Debug --toolchain ./build/Debug/generators/conan_toolchain.cmake -DCMAKE_BUILD_TYPE=Debug -G "Ninja"
+            cmake -S . -B ./build/Debug --toolchain ./build/Debug/generators/conan_toolchain.cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_UNITY_BUILD=Debug -G "Ninja"
-            cmake --build build/Debug --parallel $(nproc)
 
       - name: Clang-Tidy
         run: run-clang-tidy -p ./build/Debug/ >> tidy_result.txt
 
-      - name: Infer
-        run: infer run --compilation-database build/Debug/compile_commands.json
-
       - name: SonarQube Scan
-        if: github.head_ref == 'main' || github.ref_name == 'main'
         uses: SonarSource/sonarqube-scan-action@v7.0.0
-        with:
-          args: >
-            -Dsonar.cxx.jsonCompilationDatabase=./build/Debug/compile_commands.json
-            -Dsonar.verbose=true
         env:
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
           SONAR_HOST_URL: ${{ secrets.SONAR_HOST }}

.inferconfig

@@ -1,3 +0,0 @@
-{
-
-}

CMake/Package.cmake

@@ -22,8 +22,6 @@ function(bigfoot_create_package_lib PackagePublicDependencies PackagePrivateDepe
             ${_BF_SOURCES}
     )
 
-    target_compile_options(${PROJECT_NAME} PRIVATE ${BIGFOOT_CXX_FLAGS})
-
     target_link_libraries(${PROJECT_NAME} PUBLIC unordered_dense::unordered_dense EASTL::EASTL flatbuffers::flatbuffers rapidhash::rapidhash)
     target_link_libraries(${PROJECT_NAME} PRIVATE ${CMAKE_DL_LIBS})
 
@@ -57,9 +55,6 @@ function(bigfoot_create_package_tests ParentFolder BigfootDependencies)
             ${_BF_TEST_SOURCES}
     )
 
-    target_compile_options(${PROJECT_NAME} PRIVATE ${BIGFOOT_CXX_FLAGS})
-    target_link_options(${PROJECT_NAME} PRIVATE ${BIGFOOT_EXE_LINK_FLAGS})
-
     target_include_directories(${PROJECT_NAME} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/Include)
 
     target_link_libraries(${PROJECT_NAME} PRIVATE $<LINK_LIBRARY:WHOLE_ARCHIVE,${PackageName}>)
@@ -94,4 +89,5 @@ function(bigfoot_create_package_tests ParentFolder BigfootDependencies)
     add_dependencies(${PROJECT_NAME} ${PROJECT_NAME}Fixture)
 
     set_target_properties(${PROJECT_NAME}Fixture PROPERTIES FOLDER UtilityTargets/Tests/Bigfoot/${ParentFolder})
+
 endfunction()

CMakeLists.txt

@@ -1,6 +1,5 @@
 cmake_minimum_required(VERSION 3.24)
 
-# CMake sets this flag by default, we don't use exception in bigfoot, we can remove it
 string(REPLACE "/EHsc" "" CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
 
 project(Bigfoot VERSION 0.1.0

sonar-project.properties

@@ -9,6 +9,3 @@ sonar.tests=Bigfoot/Tests
 
 sonar.cxx.file.suffixes=.hpp,.cpp,.h
 sonar.cxx.clangtidy.reportPaths=tidy_result.txt
-sonar.cxx.infer.reportPaths=infer-out/report.json
-
-sonar.cxx.jsonCompilationDatabase.analyzeOnlyContainedFiles=True