Make step failures more specific than "bash failed"

[rotu]

Description

When a step fails, CI should report the step which failed.

Completion Criteria

Make failures report that colcon (or whatever tool) failed instead of bash.

See https://github.com/RoverRobotics-forks/rmw_cyclonedds/runs/565373542?check_suite_focus=true

[thomas-moulard]

If you click on "Invoking bash...", you can see the tool output. You can also download the raw logs.

https://help.github.com/en/actions/configuring-and-managing-workflows/managing-a-workflow-run#downloading-logs

[rotu]

I know how to dig into the data. I was calling out that the top-level error is pretty unuseful. This seems to be a negative consequence of wrapping everything in a new bash script as opposed to using ToolRunner directly.

[emersonknapp]

It would be useful to try and parse error output to get more useful data.

[rotu]

I don't think even parsing output is needed - simply not wrapping all executables in a bash shell would suffice.