ci(coverage): fix gcovr report generation

LizardByte · Oct 13, 2024 · aa516fb · aa516fb
1 parent 768cfdd
commit aa516fb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -145,9 +145,14 @@ jobs:
             --exclude-noncode-lines \
             --exclude-throw-branches \
             --exclude-unreachable-branches \
+            --verbose \
             --xml-pretty \
             -o coverage.xml
 
+      - name: Debug coverage file
+        run: |
+          cat build/coverage.xml
+
       - name: Set codecov flags
         id: codecov_flags
         run: |

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -115,7 +115,7 @@ if(CMAKE_PROJECT_NAME STREQUAL PROJECT_NAME)
         # Additional setup for coverage
         # https://gcovr.com/en/stable/guide/compiling.html#compiler-options
         #
-        if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
+        if(NOT CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
             set(CMAKE_CXX_FLAGS "-fprofile-arcs -ftest-coverage -ggdb -O0")
             set(CMAKE_C_FLAGS "-fprofile-arcs -ftest-coverage -ggdb -O0")
         endif()