Skip to content

Commit

Permalink
fix: bad substitution
Browse files Browse the repository at this point in the history
  • Loading branch information
timbru31 authored Jan 15, 2025
1 parent 6a126c9 commit 391f36c
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions .github/workflows/dockerimage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,15 @@ jobs:

- name: Build and test Docker image
run: |
VARIANT_PATH="${{ matrix.variant != 'default' && '/${{ matrix.variant }}' || '' }}"
IMAGE_PATH="${{ matrix.ruby-version }}/${{ matrix.node-version }}${VARIANT_PATH}"
VARIANT_LABEL="${{ matrix.variant != 'default' && ' ${{ matrix.variant }}' || '' }}"
if [ "${{ matrix.variant }}" = "default" ]; then
VARIANT_PATH=""
VARIANT_LABEL=""
else
VARIANT_PATH="/${{ matrix.variant }}"
VARIANT_LABEL=" ${{ matrix.variant }}"
fi
IMAGE_PATH="${{ matrix.ruby-version }}/${{ matrix.node-version }}${VARIANT_PATH}"
echo "${{ matrix.ruby-version }} with Node.js ${{ matrix.node-version }}${VARIANT_LABEL}"
# Run container and capture version outputs
Expand All @@ -37,7 +42,7 @@ jobs:
echo "::error::Expected Ruby ${{ matrix.ruby-version }} but got different version"
exit 1
fi
# Check Node version
if ! echo "$VERSIONS" | grep -q "v${{ matrix.node-version }}"; then
echo "::error::Expected Node.js ${{ matrix.node-version }} but got different version"
Expand All @@ -50,7 +55,6 @@ jobs:
permissions:
pull-requests: write
contents: write

steps:
- name: Automatically merge dependabot upgrades
uses: fastify/[email protected]
Expand Down

0 comments on commit 391f36c

Please sign in to comment.