exit code incorrectly set for some errors

Question

exit code incorrectly set for some errors

Closed this issue 4 years ago · 2 comments

There are some errors that do not have the exit code correctly set. Running locally this isn't as apparent, but when run within CI it will look like a green build when in reality there is an error.

example: https://github.com/toastdotdev/toast/pull/33/checks?check_run_id=1230126662#step:13:22

Answer 1 · 2020-10-11T05:05:34.000Z

this is due to the fact that node stderr is not currently treated as an error and needs to be. We also have to strip the warning that is currently output there out and also make sure the text output is output above the progress bars so that stderr from node doesn't fight with the progress bars (also on stderr)

Answer 2 · 2020-10-12T03:14:36.000Z

this should be fixed as of #38