Skip to content

Commit 18cff9d

Browse files
authored
fix the concurrency group of the integration workflow (#3378)
* fix the concurrency group of the integration workflow * fix: remove duplicate conditional logic from reusable workflows The reusable workflows had duplicate conditions that don't work properly in workflow_call context since github.event.pull_request is not available. The main test.yml workflow already handles these conditions correctly.
1 parent b128e31 commit 18cff9d

3 files changed

Lines changed: 1 addition & 11 deletions

File tree

.github/workflows/reusable-examples.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,6 @@ on:
1212
jobs:
1313
examples:
1414
runs-on: ubuntu-latest
15-
if: >
16-
(github.event.pull_request.draft == false &&
17-
!contains(github.event.pull_request.labels.*.name, 'ci/skip') &&
18-
!contains(github.event.pull_request.head.ref, 'datadog-api-spec/test/')) ||
19-
github.event_name == 'schedule'
2015
steps:
2116
- uses: actions/checkout@v3
2217
with:

.github/workflows/reusable-integration-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ on:
5858
required: false
5959

6060
concurrency:
61-
group: integration-${{ github.head_ref }}
61+
group: integration-go-${{ inputs.target-branch || github.head_ref }}
6262
cancel-in-progress: true
6363

6464
jobs:

.github/workflows/reusable-pre-commit.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,6 @@ env:
2626
jobs:
2727
pre-commit:
2828
runs-on: ubuntu-latest
29-
if: >
30-
(github.event.pull_request.draft == false &&
31-
!contains(github.event.pull_request.labels.*.name, 'ci/skip') &&
32-
!contains(github.event.pull_request.head.ref, 'datadog-api-spec/test/')) ||
33-
github.event_name == 'schedule'
3429
steps:
3530
- name: Get GitHub App token
3631
id: get_token

0 commit comments

Comments
 (0)