dependent-issues
dependent-issues copied to clipboard
chore: change depends status from pending to failure
Description
Changes the status of the blocked PR status from pending to failure when a PR is marked as dependent.
Motivation
This is an opinionated change, but I find that the orange pending status is not as appropriate as a failure notifier, in the merge status box. I find that most CI tools (e.g. jenkins) will mark the PR with the red failure notifier if the PR should not be merged. The orange pending notifier is usually reserved to ongoing CI runs and suggests that we don't yet know the outcome of the merge status.
Not particularly attached, just a suggestion!
Hey @johncardiologs thanks for the change. I'm okay with that change but this action worked differently for years. It's not great to change that out of a sudden.
Could you instead put this behavior behind a new optional configuration? so that the default behavior remains unchanged.
Would also love to see this get merged in, happy to help if additional work or testing needed.
@z0al lets get this merged