-
Notifications
You must be signed in to change notification settings - Fork 7
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
Catch unknown arguments and refer to help page #19
Conversation
Using fmt::runtime as a workaround there, I wasn't able to figure out which change caused the compiler to complain about command not being a constexpr.
46d65f8
to
e8843ef
Compare
Can I merge myself? |
Windows build seems to be broken.
Maybe just |
9d7eb47
to
5943d9a
Compare
Seems this was fixed upstream Long term it would probably be better to move everything to GTest so we have only one framework. |
Wrong button.. sorry. Didn't intend to close. |
5943d9a
to
14a49a9
Compare
Meh uncaught_exception is removed in C++20 |
True. Interesting that it doesn't compile. |
cb6d6ea
to
ee5389f
Compare
Updating doctest helped. I squashed the commits now so the CI will re-run, but it looked good before. |
Contains a number of updates to get it to build in separate commits.