Skip to content

Commit f54726f

Browse files
PauBadiaMclaude
andcommitted
Fix invalid GitHub Actions expression in test.yaml
Compute allowed-failures list in get-environments job using jq (where pipe operators work) instead of in a GitHub Actions expression (where they don't). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b38181d commit f54726f

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/test.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ jobs:
2525
runs-on: ubuntu-latest
2626
outputs:
2727
envs: ${{ steps.get-envs.outputs.envs }}
28+
allowed-failures: ${{ steps.get-envs.outputs.allowed-failures }}
2829
steps:
2930
- uses: actions/checkout@v5
3031
with:
@@ -45,6 +46,8 @@ jobs:
4546
}
4647
)')
4748
echo "envs=${ENVS_JSON}" | tee $GITHUB_OUTPUT
49+
ALLOWED=$(echo "${ENVS_JSON}" | jq -c 'map(select(.name | contains("pre")) | .name)')
50+
echo "allowed-failures=${ALLOWED}" | tee -a $GITHUB_OUTPUT
4851
4952
# Run tests through hatch. Spawns a separate runner for each environment defined in the hatch matrix obtained above.
5053
test:
@@ -101,5 +104,5 @@ jobs:
101104
steps:
102105
- uses: re-actors/alls-green@release/v1
103106
with:
104-
allowed-failures: ${{ toJSON(fromJSON(needs.get-environments.outputs.envs) | map(select(.name | contains("pre")) | .name)) }}
107+
allowed-failures: ${{ needs.get-environments.outputs.allowed-failures }}
105108
jobs: ${{ toJSON(needs) }}

0 commit comments

Comments
 (0)