Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UnitTests: Fix compilation error on MSVC. Update doctest to v2.4.11. #491

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

mwl4
Copy link
Contributor

@mwl4 mwl4 commented Jul 27, 2023

UnitTests fails to compile on MSVC without doctest update. Log:

Tests\Source\UnitTests\Math.cpp(44,2): error C3861: 'DOCTEST_REQUIRE': identifier not found
Tests\Source\Benchmarks\DataBinding.cpp(165,2): error C3861: 'DOCTEST_REQUIRE': identifier not found
.....

UnitTests fails to compile on msvc without update.
@mikke89 mikke89 added the build Build system and compilation label Jul 27, 2023
@mikke89 mikke89 merged commit 4504817 into mikke89:master Jul 27, 2023
13 checks passed
@mikke89
Copy link
Owner

mikke89 commented Jul 27, 2023

Thanks for the pull request. I haven't seen this issue myself, but updating the library is good to me regardless.

@mwl4 mwl4 deleted the tests_compile_error_msvc branch August 22, 2023 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build system and compilation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants