Skip to content

Commit 9f1a009

Browse files
samzongrootfs
andauthored
🔧 chore(ci): skip workflows for draft pull requests (#776)
Signed-off-by: samzong <[email protected]> Co-authored-by: Huamin Chen <[email protected]>
1 parent f6099a1 commit 9f1a009

9 files changed

+19
-7
lines changed

‎.github/workflows/docker-publish.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ concurrency:
3636
jobs:
3737
# Build job for multi-architecture Docker images
3838
build_multiarch:
39-
if: github.repository == 'vllm-project/semantic-router'
39+
if: github.repository == 'vllm-project/semantic-router' && !github.event.pull_request.draft
4040
runs-on: ubuntu-latest
4141
permissions:
4242
contents: read

‎.github/workflows/helm-publish.yml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Publish Helm Chart
22

33
on:
44
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
56
branches:
67
- main
78
paths:
@@ -22,6 +23,7 @@ env:
2223
jobs:
2324
validate-and-package:
2425
name: Validate and Package
26+
if: ${{ !github.event.pull_request.draft }}
2527
runs-on: ubuntu-latest
2628
permissions:
2729
contents: read

‎.github/workflows/integration-test-docker.yml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Integration Test [Docker Compose]
22

33
on:
44
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
56
branches:
67
- main
78
paths-ignore:
@@ -21,7 +22,7 @@ concurrency:
2122

2223
jobs:
2324
test-quickstart:
24-
if: github.repository == 'vllm-project/semantic-router'
25+
if: github.repository == 'vllm-project/semantic-router' && !github.event.pull_request.draft
2526
runs-on: ubuntu-latest
2627
timeout-minutes: 30
2728

‎.github/workflows/integration-test-dynamic-config.yml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Integration Test [Dynamic Config]
22

33
on:
44
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
56
branches:
67
- main
78
paths-ignore:
@@ -21,6 +22,7 @@ concurrency:
2122

2223
jobs:
2324
integration-test:
25+
if: ${{ !github.event.pull_request.draft }}
2426
runs-on: ubuntu-latest
2527
timeout-minutes: 180
2628

‎.github/workflows/integration-test-helm.yml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ on:
88
- 'website/**'
99
- '**/*.md'
1010
pull_request:
11+
types: [opened, synchronize, reopened, ready_for_review]
1112
branches:
1213
- main
1314
paths-ignore:
@@ -30,6 +31,7 @@ jobs:
3031
# Lint and validate Helm chart
3132
lint-chart:
3233
name: Lint Helm Chart
34+
if: ${{ !github.event.pull_request.draft }}
3335
runs-on: ubuntu-latest
3436
steps:
3537
- name: Checkout code

‎.github/workflows/integration-test-k8s.yml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Integration Test [Kubernetes]
22

33
on:
44
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
56
branches:
67
- main
78
paths-ignore:
@@ -21,6 +22,7 @@ concurrency:
2122

2223
jobs:
2324
integration-test:
25+
if: ${{ !github.event.pull_request.draft }}
2426
runs-on: ubuntu-latest
2527
timeout-minutes: 75
2628
strategy:

‎.github/workflows/owner-notification.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@ name: Owner Notification
22

33
on:
44
pull_request_target:
5-
types: [assigned, opened, reopened, synchronize]
5+
types: [assigned, opened, reopened, synchronize, ready_for_review]
66

77
jobs:
88
notify-owners:
9-
if: ${{ github.repository == 'vllm-project/semantic-router' }}
9+
if: ${{ github.repository == 'vllm-project/semantic-router' && !github.event.pull_request.draft }}
1010
runs-on: ubuntu-latest
1111
permissions:
1212
contents: read

‎.github/workflows/precommit-publish.yml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ on:
55
paths:
66
- 'Dockerfile.precommit'
77
pull_request:
8+
types: [opened, synchronize, reopened, ready_for_review]
89
paths:
910
- 'Dockerfile.precommit'
1011

@@ -14,7 +15,7 @@ concurrency:
1415

1516
jobs:
1617
build_and_push:
17-
if: github.repository == 'vllm-project/semantic-router'
18+
if: github.repository == 'vllm-project/semantic-router' && !github.event.pull_request.draft
1819
runs-on: ubuntu-latest
1920
permissions:
2021
contents: read

‎.github/workflows/test-and-build.yml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ name: Test And Build
22

33
on:
44
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
56
branches:
67
- main
78
push:
@@ -24,13 +25,14 @@ jobs:
2425
test-and-build:
2526
needs: changes
2627
if: >-
27-
${{ github.event_name == 'schedule'
28+
${{ !github.event.pull_request.draft
29+
&& (github.event_name == 'schedule'
2830
|| needs.changes.outputs.core == 'true'
2931
|| needs.changes.outputs.helm == 'true'
3032
|| needs.changes.outputs.e2e == 'true'
3133
|| needs.changes.outputs.docker == 'true'
3234
|| needs.changes.outputs.make == 'true'
33-
|| needs.changes.outputs.ci == 'true' }}
35+
|| needs.changes.outputs.ci == 'true') }}
3436
runs-on: ubuntu-latest
3537

3638
steps:

0 commit comments

Comments
 (0)