Carry through Exit Code

[bdowne01]

Hi, thanks for such a useful action!

After trying this out, I'm a little confused on how to properly fail the workflow job if one of the terraform steps fail.

We're passing continue-on-error: true to the terraform step, like the example shows. It seems the commenter step(s) should exit non-zero after posting the comment as to stop processing subsequent job steps, but the commenter exits zero even when it was handed a non-zero from the terraform wrapper.

I feel like I'm missing something obvious but the readme states

In English: "Always run this step, but only on a pull request and only when the previous step succeeds or fails...and then stop the build.

What trigger should be used stop the build?

Thanks again!

[JensRantil]

Completely agree with this one. I think the exit code should carry through.