Add on-failure option to comment-summary-in-pr setting by sgmurphy · Pull Request #540 · actions/dependency-review-action (original) (raw)

@sgmurphy This idea looks good, thank you 👍. It will existing behavior (true/false values) in order to keep supporting users. I'm not sure how you'd like to proceed, an idea:

1. We check the input value and if it's a boolean we map true/false to the new string values.
2. Update the docs with a mention to this change.
3. We do a new release bumping the minor version number to 3.1, minimizing impact to existing users with pinned versions.

I can help with (2) and (3), but would like your input on (1), how do you think we should handle this transition for existing users without breaking their existing setups?