codecov-action icon indicating copy to clipboard operation
codecov-action copied to clipboard

Does not honor return code from bash script (fail_ci_if_error / -Z)

Open blueyed opened this issue 4 years ago • 3 comments

With fail_ci_if_error = true -Z is passed to the bash script, but apparently its non-zero return code is not honored / turned into a CI failure.

(Note that due to https://github.com/codecov/codecov-bash/issues/415 it returns non-zero currently always, i.e. every GitHub Action that uses codecov-action with fail_ci_if_error should be affected by this.)

blueyed avatar Apr 19 '21 11:04 blueyed

@blueyed just wanted to sync back that we pushed the fix from bash to the actions repo now (1.5.1). Is this good to close?

thomasrockhu avatar Jun 08 '21 16:06 thomasrockhu

@thomasrockhu codecov-bash is fixed, thanks.

But the issue here is that codecov-action did not fail, although codecov-bash did. But this then also was only triggered when using -X s3, which the action does not use.

However, from quickly skimming the code it seemed like that might still be the case though anyway - that's something you should check in the code, and/or test it even.

blueyed avatar Jun 08 '21 17:06 blueyed

Related #354

RA80533 avatar Jun 11 '21 02:06 RA80533

I believe this has been fixed, feel free to re-open if I'm wrong here.

thomasrockhu-codecov avatar Feb 28 '23 15:02 thomasrockhu-codecov