Skip to content

Commit 6e585a3

Browse files
committed
require approvals on any PRs from forks regardless of the trigger
1 parent 1f0f74b commit 6e585a3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/test-warehouse.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ env:
5353
TESTS_DIR: ${{ github.workspace }}/dbt-data-reliability/integration_tests
5454

5555
jobs:
56-
# PRs from forks require approval, specifically with the "pull_request_target" event as it contains repo secrets.
56+
# PRs from forks require approval
5757
check-if-requires-approval:
5858
runs-on: ubuntu-latest
5959
outputs:
@@ -62,7 +62,7 @@ jobs:
6262
- name: Set requires approval output
6363
id: set-output
6464
run: |
65-
if [[ "${{ github.event_name }}" == "pull_request_target" && "${{ github.event.pull_request.head.repo.full_name }}" != "${{ github.repository }}" ]]; then
65+
if [[ "${{ github.event.pull_request.head.repo.full_name }}" != "${{ github.repository }}" ]]; then
6666
echo "requires_approval=true" >> $GITHUB_OUTPUT
6767
else
6868
echo "requires_approval=false" >> $GITHUB_OUTPUT

0 commit comments

Comments
 (0)