-
Notifications
You must be signed in to change notification settings - Fork 26
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
Bail on first error #27
Comments
+1 I think cutest should keep the default behaviour of failing fast. It's very useful while developing. Running all tests regardless of there being failures would be very useful for running automated tests on a CI server for example. |
Cool. Yeah, I agree with you. And keeping the default behaviour makes the change backwards compatible. I'll invert the flag (or how should it be called then?). |
@djanowski any comments here? I can just flip the flag if you're ok going ahead with #28. |
@gtramontina The flag can be |
Hard to come up with the right name. I was thinking maybe |
I think the usage string (in the executable) should be clearer, though. |
We already have I'm still inclined for |
Currently
cutest
exits when the first error occur. Would you be interested in adding a flag-b
to make that optional? The reason is that sometimes it is useful to get all errors at once.The text was updated successfully, but these errors were encountered: