-
Notifications
You must be signed in to change notification settings - Fork 17
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
Update to Catch2 version 3.0.1. #437
Conversation
You might want to mention in the "Dependencies" section of |
Could you use more descriptive titles for PRs please? When I auto-generate release notes it uses the PR title, and "Close 434" isn't very informative in release notes. I'd rather see something like "Update to catch2 3.0.1." |
We should probably also add |
I think they moved from a header only library to a normal library in version 3, so all the tests need to be linked with catch2 now. |
Spack has Catch2 package. It uses 3.0.1 version by default.