Skip to content

Commit

Permalink
Moving from pull_request_target back to pull_request for now (appsmit…
Browse files Browse the repository at this point in the history
  • Loading branch information
mohanarpit authored and hetunandu committed Mar 17, 2021
1 parent 0051c3f commit 482306f
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/client.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
- 'app/client/**'
- '!app/client/cypress/manual_TestSuite/**'

pull_request_target:
pull_request:
branches: [release, master]
paths:
- 'app/client/**'
Expand All @@ -33,7 +33,7 @@ jobs:
steps:
# Checkout the code
- name: Checkout the merged commit from PR and base branch
if: github.event_name == 'pull_request_target'
if: github.event_name == 'pull_request'
uses: actions/checkout@v2
with:
fetch-depth: 0
Expand Down Expand Up @@ -142,7 +142,7 @@ jobs:
steps:
# Checkout the code
- name: Checkout the merged commit from PR and base branch
if: github.event_name == 'pull_request_target'
if: github.event_name == 'pull_request'
uses: actions/checkout@v2
with:
ref: refs/pull/${{ github.event.pull_request.number }}/merge
Expand Down Expand Up @@ -262,7 +262,7 @@ jobs:
group: "Electrons on Github Action"
spec: "cypress/integration/Smoke_TestSuite/**/*.js"
working-directory: app/client
# tag will be either "push" or "pull_request_target"
# tag will be either "push" or "pull_request"
tag: ${{ github.event_name }}
env: "NODE_ENV=development"

Expand All @@ -285,7 +285,7 @@ jobs:
steps:
# Checkout the code
- name: Checkout the merged commit from PR and base branch
if: github.event_name == 'pull_request_target'
if: github.event_name == 'pull_request'
uses: actions/checkout@v2
with:
ref: refs/pull/${{ github.event.pull_request.number }}/merge
Expand Down

0 comments on commit 482306f

Please sign in to comment.