Skip to content

Commit 12359d5

Browse files
committed
Fix == true comparisons in quicktest
1 parent 65d3a8a commit 12359d5

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

.github/workflows/quicktest.yaml

+8-8
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ env:
4242
REFERENCE_DIR: /tmp/reference
4343

4444
jobs:
45-
build-docker-latest:
45+
build-docker:
4646
name: 'Check and build the current docker image'
4747
runs-on: ubuntu-latest
4848
timeout-minutes: 180
@@ -117,10 +117,10 @@ jobs:
117117
# this may also need adaptations in the test script at the bottom
118118
echo "EXTRA_ARGS=" >> $GITHUB_OUTPUT
119119
- name: Checkout repository
120-
if: ${{ steps.parse.outputs.CONTINUE && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
120+
if: ${{ steps.parse.outputs.CONTINUE == 'true' && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
121121
uses: actions/checkout@v4
122122
- name: Get the FreeSurfer version
123-
if: ${{ steps.parse.outputs.CONTINUE && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
123+
if: ${{ steps.parse.outputs.CONTINUE == 'true' && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
124124
shell: bash
125125
id: parse-version
126126
run: |
@@ -139,7 +139,7 @@ jobs:
139139
echo "FASTSURFER_HOME=$(pwd)"
140140
} > $GITHUB_OUTPUT
141141
- uses: Deep-MI/FastSurfer/.github/actions/build-docker@dev
142-
if: ${{ steps.parse.outputs.CONTINUE && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
142+
if: ${{ steps.parse.outputs.CONTINUE == 'true' && steps.parse.outputs.DOCKER_IMAGE == 'build-cached' }}
143143
# This action needs the "full" checkout (located at ${{ inputs.fastsurfer-home }})
144144
with:
145145
fastsurfer-home: ${{ steps.parse-version.outputs.FASTSURFER_HOME }}
@@ -148,10 +148,10 @@ jobs:
148148
freesurfer-build-image: "${{ steps.parse-version.outputs.FS_BUILD_IMAGE }}"
149149
fastsurfer:
150150
name: 'Run FastSurfer on sample images'
151-
needs: build-docker-latest
151+
needs: build-docker
152152
runs-on: ubuntu-latest
153153
timeout-minutes: 180
154-
if: needs.build-docker-latest.outputs.continue
154+
if: needs.build-docker.outputs.continue == 'true'
155155
strategy:
156156
# the following matrix strategy will result in one run per subject as matrix is "one-dimensional".
157157
# Additional parameters under "include" are then added as additional (dependent) information
@@ -175,8 +175,8 @@ jobs:
175175
file-extension: ${{ matrix.file-extension }}
176176
image-href: ${{ secrets[matrix.image-key] }}
177177
license: ${{ secrets.QUICKTEST_LICENSE }}
178-
docker-image: ${{ needs.build-docker-latest.outputs.docker-image }}
179-
extra-args: ${{ needs.build-docker-latest.outputs.extra-args }}
178+
docker-image: ${{ needs.build-docker.outputs.docker-image }}
179+
extra-args: ${{ needs.build-docker.outputs.extra-args }}
180180
tests:
181181
name: 'Download data and perform tests'
182182
needs: fastsurfer

0 commit comments

Comments
 (0)