Skip to content

Commit 1eba222

Browse files
ci: updated CI to not run for draft PRs (#290)
1 parent f027516 commit 1eba222

5 files changed

Lines changed: 34 additions & 1 deletion

File tree

.github/workflows/accuracy_performance.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ name: PR Evaluation
66

77
on:
88
pull_request:
9+
types: [opened, synchronize, reopened, ready_for_review]
910
branches:
1011
- main
1112
paths:
@@ -17,6 +18,10 @@ on:
1718
- 'benchmarks/**'
1819
workflow_dispatch:
1920

21+
concurrency:
22+
group: ${{ github.workflow }}-${{ github.ref }}
23+
cancel-in-progress: true
24+
2025
permissions:
2126
contents: read
2227
packages: write
@@ -28,6 +33,7 @@ jobs:
2833
# All downstream jobs pull these images instead of rebuilding.
2934
# ---------------------------------------------------------------------------
3035
build:
36+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
3137
name: Build CI images
3238
runs-on: ubuntu-latest
3339
outputs:

.github/workflows/correctness.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ on:
1313
- 'asap-common/sketch-core/**'
1414
- '.github/workflows/correctness.yml'
1515
pull_request:
16+
types: [opened, synchronize, reopened, ready_for_review]
1617
branches: [ main ]
1718
paths:
1819
- 'asap-common/tests/**'
@@ -21,9 +22,14 @@ on:
2122
- '.github/workflows/correctness.yml'
2223
workflow_dispatch:
2324

25+
concurrency:
26+
group: ${{ github.workflow }}-${{ github.ref }}
27+
cancel-in-progress: true
28+
2429
jobs:
2530
# ── 1. Cross-language token matching (Python vs Rust) ──────────────────────
2631
cross-language-token-matching:
32+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
2733
name: Cross-language PromQL token matching
2834
runs-on: ubuntu-latest
2935
steps:
@@ -82,6 +88,7 @@ jobs:
8288

8389
# ── 2. Cross-language serialised pattern comparison ────────────────────────
8490
cross-language-pattern-serialisation:
91+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
8592
name: Cross-language pattern serialisation
8693
runs-on: ubuntu-latest
8794
steps:
@@ -136,6 +143,7 @@ jobs:
136143

137144
# ── 3. Rust pattern-matching unit tests ────────────────────────────────────
138145
rust-pattern-matching:
146+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
139147
name: Rust pattern matching unit tests
140148
runs-on: ubuntu-latest
141149
steps:

.github/workflows/docker.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ on:
55
tags:
66
- 'v[0-9]+.[0-9]+.[0-9]+*'
77
pull_request:
8+
types: [opened, synchronize, reopened, ready_for_review]
89
branches: [main]
910
paths:
1011
- 'asap-common/installation/**'
@@ -21,12 +22,17 @@ on:
2122
required: true
2223
default: 'dev'
2324

25+
concurrency:
26+
group: ${{ github.workflow }}-${{ github.ref }}
27+
cancel-in-progress: true
28+
2429
permissions:
2530
packages: write
2631
contents: read
2732

2833
jobs:
2934
build-images:
35+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
3036
runs-on: ubuntu-latest
3137
steps:
3238
- name: Checkout code

.github/workflows/python.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ on:
1212
- 'asap-common/dependencies/py/**'
1313
- '.github/workflows/python.yml'
1414
pull_request:
15+
types: [opened, synchronize, reopened, ready_for_review]
1516
branches: [ main ]
1617
paths:
1718
- 'asap-summary-ingest/**'
@@ -23,12 +24,17 @@ on:
2324
- '.github/workflows/python.yml'
2425
workflow_dispatch:
2526

27+
concurrency:
28+
group: ${{ github.workflow }}-${{ github.ref }}
29+
cancel-in-progress: true
30+
2631
permissions:
2732
contents: read
2833
pull-requests: read
2934

3035
jobs:
3136
detect-changes:
37+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
3238
runs-on: ubuntu-latest
3339
outputs:
3440
summary_ingest: ${{ steps.filter.outputs.summary_ingest }}

.github/workflows/rust.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ on:
1212
- 'Cargo.lock'
1313
- '.github/workflows/rust.yml'
1414
pull_request:
15+
types: [opened, synchronize, reopened, ready_for_review]
1516
branches: [ main ]
1617
paths:
1718
- 'asap-query-engine/**'
@@ -23,11 +24,16 @@ on:
2324
- '.github/workflows/rust.yml'
2425
workflow_dispatch:
2526

27+
concurrency:
28+
group: ${{ github.workflow }}-${{ github.ref }}
29+
cancel-in-progress: true
30+
2631
env:
2732
CARGO_TERM_COLOR: always
2833

2934
jobs:
3035
format-and-lint:
36+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
3137
runs-on: ubuntu-latest
3238
steps:
3339
- uses: actions/checkout@v4
@@ -64,6 +70,7 @@ jobs:
6470
RUSTC_WRAPPER: sccache
6571

6672
test:
73+
if: github.event_name != 'pull_request' || github.event.pull_request.draft == false
6774
runs-on: ubuntu-latest
6875
steps:
6976
- uses: actions/checkout@v4
@@ -108,4 +115,4 @@ jobs:
108115
uses: docker/setup-buildx-action@v3
109116

110117
- name: Build Docker image
111-
run: docker build -f asap-query-engine/Dockerfile -t sketchdb-queryengine-rust:latest .
118+
run: docker build -f asap-query-engine/Dockerfile -t sketchdb-queryengine-rust:latest .

0 commit comments

Comments
 (0)