Skip to content

Commit 356a51b

Browse files
authored
Update makefile.yml
Trying to only run AWS stuff if PR (not sure what happens if it's a push to an existing PR, see https://stackoverflow.com/questions/66205887/only-run-github-actions-step-if-not-a-pull-request)
1 parent 7565709 commit 356a51b

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.github/workflows/makefile.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,12 @@ jobs:
2727
aws-region: us-east-1
2828
aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
2929
aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
30+
if: github.event_name != 'pull_request'
3031

3132
- name: S3 test
3233
run: |
3334
aws s3 ls s3://pyret-horizon/new-horizons
35+
if: github.event_name != 'pull_request'
3436

3537
- name: Install Chrome and ChromeDriver
3638
run: |
@@ -117,4 +119,4 @@ jobs:
117119
- name: Copy cpo-main and related JS to S3
118120
run: |
119121
aws s3 sync ./build/web/js s3://pyret-horizon/horizon-action-test --acl public-read --exclude "*" --include "cpo-main.jarr.gz.js" --content-encoding gzip
120-
aws s3 sync ./build/web/js s3://pyret-horizon/horizon-action-test --acl public-read --exclude "cpo-main.jarr.js.gz"
122+
aws s3 sync ./build/web/js s3://pyret-horizon/horizon-action-test --acl public-read --exclude "cpo-main.jarr.js.gz"

0 commit comments

Comments
 (0)