Skip to content

Commit 14ea423

Browse files
authored
Merge pull request #1785 from vespa-engine/laura/fix-sample-app-json
invert our check for master branch for dryrun and echo since empty string is falsy
2 parents f792c69 + 29dd573 commit 14ea423

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/update-cloud-samples.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ jobs:
142142
- name: Push Zip and JSON to S3
143143
env:
144144
# Add `--dryrun` if not on "push" event on default branch.
145-
AWS_S3_OPTIONS: --color=on --no-progress ${{ github.ref_name == 'master' && '' || ' --dryrun' }}
145+
AWS_S3_OPTIONS: --color=on --no-progress ${{ github.ref_name != 'master' && ' --dryrun' || '' }}
146146
run: |
147147
# Not an app, but artifact from the setup job.
148148
mv ./apps/console-json ./console-json
@@ -152,6 +152,6 @@ jobs:
152152
153153
- name: Invalidate Cloudfront Cache
154154
env:
155-
ECHO: ${{ (github.event_name == 'push' && github.ref_name == 'master') && '' || 'echo' }}
155+
ECHO: ${{ github.ref_name != 'master' && ' echo' || '' }}
156156
run: |
157157
$ECHO aws cloudfront create-invalidation --distribution-id "${AWS_CLOUDFRONT_DISTRIBUTION_ID}" --paths "${S3_UPLOAD_PREFIX}/*" --output text

0 commit comments

Comments
 (0)