-
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
Check fails, stdout and stderr #389
Comments
This is not an error from the |
This makes sense. But I see more that as a workflow error than a computation one. This makes me write weird lines of code : ugly tests . |
I suspect you are doing something ... odd from our point of view: the entire idea with |
Using parallelism, my script speeds up the search for regression when I modify my code. If I have an unexpected failure, I then work on each failing test on a case-by-case basis. PS: in the end, once everything is fixed, I use |
We do that too :) |
@zauguin You are the expert in this area - what's your take? |
When check fails, you don't use the
stdout
stream. It should be better to indicate the failure "Check failed with <>" inside thestderr
stream. Doing this allow extra tools to catch only checking that gives an error without having to parse all thestdout
stream.The text was updated successfully, but these errors were encountered: