Niels Lohmann 819838839d Include doctest as a SYSTEM directory instead of disabling warnings for all tests ...

test_main added -Wno-deprecated and -Wno-float-equal as PUBLIC compile
options for every non-MSVC compiler, so they were applied to every
translation unit, library headers included, and silenced the CI
warnings meant to check the library's own -Wfloat-equal pragmas. The
only code that actually needed the suppression was the vendored
doctest.h, which was included as a normal (non-SYSTEM) directory.

Include thirdparty/doctest as SYSTEM for test_main, matching what
tests/abi/CMakeLists.txt already does, and drop the two suppressions
from both targets. Verified locally that unit-comparison,
unit-conversions and unit-constructor1 compile clean with
-Werror -Weverything and doctest as -isystem, and that CMake still
configures with JSON_BuildTests=ON.

Part of #5725

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

2026-09-30 09:54:07 +02:00

..

config

Make JSON_STRICT_NUL_HANDLING part of the ABI tag (#5560)

2026-09-25 21:54:11 +02:00

diag

📄 adjust year (#5044)

2026-01-01 20:00:39 +01:00

include/nlohmann

Fix spellcheck issue (#4173)

2023-10-04 15:24:38 +02:00

inline_ns

📄 adjust year (#5044)

2026-01-01 20:00:39 +01:00

CMakeLists.txt

Include doctest as a SYSTEM directory instead of disabling warnings for all tests

2026-09-30 09:54:07 +02:00

main.cpp

📄 adjust year (#5044)

2026-01-01 20:00:39 +01:00