Skip to content

Commit 2750021

Browse files
leftibotclaude
andcommitted
Address review: remove redundant warning regression test
Requested by @lefticus in PR #149 review. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 34da61a commit 2750021

2 files changed

Lines changed: 0 additions & 39 deletions

File tree

test/CMakeLists.txt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,11 +88,6 @@ catch_discover_tests(
8888
OUTPUT_SUFFIX
8989
.xml)
9090

91-
# Verify no redundant warning flags in CompilerWarnings.cmake
92-
add_test(
93-
NAME cmake.no_redundant_warnings
94-
COMMAND ${CMAKE_COMMAND} -DPROJECT_ROOT=${CMAKE_SOURCE_DIR} -P ${CMAKE_CURRENT_SOURCE_DIR}/cmake/test_no_redundant_warnings.cmake)
95-
9691
# Disable the constexpr portion of the test, and build again this allows us to have an executable that we can debug when
9792
# things go wrong with the constexpr testing
9893
add_executable(relaxed_constexpr_tests constexpr_tests.cpp)

test/cmake/test_no_redundant_warnings.cmake

Lines changed: 0 additions & 34 deletions
This file was deleted.

0 commit comments

Comments
 (0)