Skip to content

Commit d21f0f6

Browse files
committed
change job name
1 parent ad8124a commit d21f0f6

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

.github/workflows/test-warehouse.yml

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

5555
jobs:
56-
detect-fork:
56+
# PRs from forks require approval, specifically with the "pull_request_target" event as it contains repo secrets.
57+
check-if-requires-approval:
5758
runs-on: ubuntu-latest
5859
outputs:
5960
requires_approval: ${{ steps.set-output.outputs.requires_approval }}
6061
steps:
61-
- name: Set fork output
62+
- name: Set requires approval output
6263
id: set-output
6364
run: |
6465
if [[ "${{ github.event_name }}" == "pull_request_target" && "${{ github.event.pull_request.head.repo.full_name }}" != "${{ github.repository }}" ]]; then
@@ -69,9 +70,8 @@ jobs:
6970
7071
test:
7172
runs-on: ubuntu-latest
72-
needs: [detect-fork]
73-
# PRs from forks require approval - so we use an environment that requires an approver from the team.
74-
environment: ${{ (needs.detect-fork.outputs.requires_approval == 'true' && 'elementary_test_env') || '' }}
73+
needs: [check-if-requires-approval]
74+
environment: ${{ (needs.check-if-requires-approval.outputs.requires_approval == 'true' && 'elementary_test_env') || '' }}
7575
concurrency:
7676
# This is what eventually defines the schema name in the data platform.
7777
group: tests_${{ inputs.warehouse-type }}_dbt_${{ inputs.dbt-version }}_${{ github.head_ref || github.ref_name }}

0 commit comments

Comments
 (0)