Catch2
Catch2 copied to clipboard
Introduce CATCH_CONFIG_PREFIX_MESSAGES to only prefix a few logging related macros.
Description
The macro CATCH_CONFIG_PREFIX_ALL allows to prefix all of catch2's macros with CATCH_. This commit adds some granularity by introducing a new macro: CATCH_CONFIG_PREFIX_MESSAGES. It only prefixes a few debug message macros.
Why? It's rather usual that logging frameworks already use INFO/WARN macros themselves. This allows a user to avoid clashes without having to use the prefix everywhere else.
GitHub Issues
Codecov Report
Merging #2544 (14b2fee) into devel (6e79e68) will decrease coverage by
0.01%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## devel #2544 +/- ##
==========================================
- Coverage 91.20% 91.19% -0.01%
==========================================
Files 192 192
Lines 7843 7843
==========================================
- Hits 7153 7152 -1
- Misses 690 691 +1
Would be nice if you can add it also here: expand_template( expand_template( to ensure the Bazel build does not break.
Would be nice if you can add it also here: expand_template( expand_template( to ensure the Bazel build does not break.
Like so?
@horenmar Any comments regarding this PR?