BenchmarkDotNet
BenchmarkDotNet copied to clipboard
Always fail if dotnet command failed
Fixes #2533
@adamsitnik PTAL
The "the exit code might be 1 due to warnings" seems like something that might be worth keeping, maybe limit it to certain ranges of exit codes, or add some sort of command line switch like '--ignoreexitcode' so that people who were relying on the old behavior aren't stuck?
I don't know the specifics around exit code 1 for warnings, but that file is quite old, so I'm thinking that may have only been an issue with early Core sdks (which we no longer support). I wasn't able to get that result with latest sdk.