Skip to content

Commit 1202708

Browse files
Merge pull request #3454 from cal-itp/fix/erroneous-maps-app-deploys
fix(ci): only deploy maps app previews on relevant pull requests
2 parents 87e570d + 16f023a commit 1202708

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

.github/workflows/deploy-apps-maps.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,14 @@ name: Build and publish apps/maps
22

33
on:
44
push:
5+
branches:
6+
- 'main'
57
paths:
8+
- '.github/workflows/deploy-apps-maps.yml'
69
- 'apps/maps/**'
710
pull_request:
811
paths:
12+
- '.github/workflows/deploy-apps-maps.yml'
913
- 'apps/maps/**'
1014
workflow_dispatch:
1115

@@ -34,7 +38,7 @@ jobs:
3438

3539
# Preview on PRs
3640
- name: Deploy apps/maps preview to Netlify
37-
if: ${{ github.ref != 'refs/heads/main' }}
41+
if: ${{ github.event_name == 'pull_request' }}
3842
run: |
3943
npm install -g netlify-cli
4044
netlify deploy --site=embeddable-maps-calitp-org --dir=apps/maps/build --alias=${GITHUB_REPOSITORY#*/}-${PR_NUMBER}

0 commit comments

Comments
 (0)