re-actors/alls-green

[TODO] Implement `allow-skips`

Closed this issue · 5 comments

webknjaz commented

This has been suggested by @pradyunsg.

Use-case: workflows that run jobs conditionally, depending on the files changed in associated PRs, for example — https://github.com/pypa/pip/blob/bbc7021/.github/workflows/ci.yml#L29-L49, using a change detection action like dorny/paths-filter.

webknjaz commented

@pradyunsg mind checking out the change?
3a2de12
https://github.com/webknjaz/check-gha-test/runs/4515033319?check_suite_focus=true#step:3:59

It is currently testable via the unstable branch:

- uses: re-actors/alls-green@unstable/v1
  id: check
  with:
    allowed-failures: failing-job0
    allowed-skips: failing-job1, skipped-job
    jobs: ${{ toJSON(needs) }}
webknjaz commented

Due to my impatience, I've published this under a stable release v1.1.0.

  • 👍1
pradyunsg commented

The behaviour looks good to me!

  • 🎉1
webknjaz commented

Perfect! @pradyunsg should I make a PR in pip?

pradyunsg commented

Yes!