-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Use Catch for unit tests #3232
Comments
Hi @nlohmann, just curious, as I am somewhat involved with doctest now :) |
This was the same reason for me to switch to Doctest. I don't really want to switch back to Catch, but I was afraid that Doctest somehow stalled as they were looking for maintainers (which seems to have been solved now). I did not compare the compilation times yet - the Catch branch #3236 does not work right now, because Catch triggers several compiler warnings. If the newest Doctest release 2.4.8 works, I will close this issue and hope to not touch the test suite again for a long time :) |
Closed. Doctest development continues. |
The latest Doctest release added a regression with MSVC 2015 (see #3228 and doctest/doctest#573). Furthermore, Doctest is looking for maintainers (doctest/doctest#554) and the future of the project is unclear. Meanwhile, Catch seems to be actively developed.
The text was updated successfully, but these errors were encountered: