Skip to content

Commit 7ae36aa

Browse files
authored
fix(ci): revert usage of pull_request_target trigger (#5903)
* remove pack-and-comment workflow STENCIL-1363 * revert usage of pull_request_target trigger
1 parent 3def2b7 commit 7ae36aa

File tree

3 files changed

+1
-106
lines changed

3 files changed

+1
-106
lines changed

.github/workflows/build.yml

-5
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,6 @@ jobs:
1212
steps:
1313
- name: Checkout Code
1414
uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
15-
with:
16-
# the pull_request_target event will consider the HEAD of `main` to be the SHA to use.
17-
# attempt to use the SHA associated with a pull request and fallback to HEAD of `main`
18-
ref: ${{ github.event_name == 'pull_request_target' && format('refs/pull/{0}/merge', github.event.number) || '' }}
19-
persist-credentials: false
2015

2116
- name: Get Core Dependencies
2217
uses: ./.github/workflows/actions/get-core-dependencies

.github/workflows/pack-and-comment.yml

-83
This file was deleted.

.github/workflows/tech-debt-burndown.yml

+1-18
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ name: Tech Debt Burndown
77
# progress on them to each PR.
88

99
on:
10-
pull_request_target:
10+
pull_request:
1111
branches:
1212
- '**'
1313

@@ -27,12 +27,6 @@ jobs:
2727

2828
- name: Checkout PR branch
2929
uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
30-
with:
31-
# the pull_request_target event will consider the HEAD of `main` to be the SHA to use.
32-
# attempt to use the SHA associated with a pull request and fallback to HEAD of `main`
33-
ref: ${{ github.event_name == 'pull_request_target' && format('refs/pull/{0}/merge', github.event.number) || '' }}
34-
persist-credentials: false
35-
if: ${{ matrix.branch == 'pr' }}
3630

3731
- name: Get Core Dependencies
3832
uses: ./.github/workflows/actions/get-core-dependencies
@@ -65,12 +59,6 @@ jobs:
6559

6660
- name: Checkout PR branch
6761
uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
68-
with:
69-
# the pull_request_target event will consider the HEAD of `main` to be the SHA to use.
70-
# attempt to use the SHA associated with a pull request and fallback to HEAD of `main`
71-
ref: ${{ github.event_name == 'pull_request_target' && format('refs/pull/{0}/merge', github.event.number) || '' }}
72-
persist-credentials: false
73-
if: ${{ matrix.branch == 'pr' }}
7462

7563
- name: Install ts-prune
7664
run: npm install ts-prune@0.10.3
@@ -91,11 +79,6 @@ jobs:
9179
steps:
9280
- name: Checkout current branch
9381
uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
94-
with:
95-
# the pull_request_target event will consider the HEAD of `main` to be the SHA to use.
96-
# attempt to use the SHA associated with a pull request and fallback to HEAD of `main`
97-
ref: ${{ github.event_name == 'pull_request_target' && format('refs/pull/{0}/merge', github.event.number) || '' }}
98-
persist-credentials: false
9982

10083
- name: Get Core Dependencies
10184
uses: ./.github/workflows/actions/get-core-dependencies

0 commit comments

Comments
 (0)