Skip to content

Commit

Permalink
ci: Fix job skips
Browse files Browse the repository at this point in the history
  • Loading branch information
max-sixty committed Aug 23, 2023
1 parent 688d325 commit fad394b
Showing 1 changed file with 24 additions and 4 deletions.
28 changes: 24 additions & 4 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -465,17 +465,37 @@ jobs:
- name: Decide whether the needed jobs succeeded or failed
uses: re-actors/alls-green@release/v1
with:
jobs: ${{ toJSON(needs) }}
# We don't include `check-links-markdown`, since occasionally we'll want to merge
# something which temporarily fails that, such as if we're changing the
# location of a file in this repo which is linked to.
#
# We're currently including `nightly` because I'm not sure whether
# it's always reliable; e.g. `cargo-audit`
allowed-failures: check-links-markdown, nightly
# Easier than writing out every job — and it's our own rules deciding
# whether jobs are skipped or not.
allowed-skips: ${{ toJSON(needs) }}
jobs: ${{ toJSON(needs) }}
# We skip jobs deliberately, so we are OK if any are skipped.
#
# Copy-pasted from `needs`, since it needs to be a json list, so `${{
# toJSON(needs) }}` (which is a map) doesn't work.
# https://github.com/re-actors/alls-green/issues/23
allowed-skips: |
[
"build-web",
"check-links-book",
"check-links-markdown",
"lint-megalinter",
"nightly",
"publish-web",
"test-dotnet",
"test-elixir",
"test-java",
"test-js",
"test-lib",
"test-php",
"test-python",
"test-rust",
"test-rust-main"
]
build-prqlc:
runs-on: ${{ matrix.os }}
Expand Down

0 comments on commit fad394b

Please sign in to comment.