Skip to content

Commit 01b4d70

Browse files
author
Christos Konstantinos Matzoros
committed
Small corrections
1 parent 7f344d9 commit 01b4d70

2 files changed

Lines changed: 3 additions & 9 deletions

File tree

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
run: cmake --build . -j$(nproc)
4040

4141
# -------------------------------------------------
42-
# 2. FULL BUILD + TEST + COVERAGE + DOCS (RelWithDebInfo + Eigen)
42+
# 2. FULL BUILD + TEST + SIMPLE DAG RUN + COVERAGE + DOCS (RelWithDebInfo + Eigen)
4343
# -------------------------------------------------
4444
test_and_run:
4545
name: Build, Test & Coverage (RelWithDebInfo + Eigen)

CMakeLists.txt

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,21 +25,15 @@ set(CMAKE_CXX_STANDARD 17)
2525
set(CMAKE_CXX_STANDARD_REQUIRED ON)
2626
set(CMAKE_CXX_EXTENSIONS OFF) # Ensure C++17 standard is strictly enforced
2727

28-
# --- Code Coverage Configuration ---
28+
# --- Code Coverage (optional) ---
2929
option(ENABLE_COVERAGE "Enable code coverage flags" OFF)
3030

3131
if(ENABLE_COVERAGE AND CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
32-
message(STATUS "Code coverage enabled: adding --coverage flags")
32+
message(STATUS "Code coverage enabled")
3333
add_compile_options(--coverage -O0 -g)
3434
add_link_options(--coverage)
3535
endif()
3636

37-
# --- Override RelWithDebInfo flags when coverage is enabled ---
38-
if(ENABLE_COVERAGE AND CMAKE_BUILD_TYPE STREQUAL "RelWithDebInfo")
39-
message(STATUS "Forcing Debug-like flags for RelWithDebInfo due to coverage")
40-
set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-O0 -g --coverage")
41-
endif()
42-
4337
# Configure git hooks
4438
add_subdirectory(.githooks)
4539

0 commit comments

Comments
 (0)