From 1d870111a88673b04558695c26001f648c8461c4 Mon Sep 17 00:00:00 2001 From: Nikolay Petrov Date: Fri, 23 Oct 2026 15:52:35 -0700 Subject: [PATCH 1/2] ci: keep main runs, drop a dead flag, tighten path filters - cancel-in-progress only for pull_request events, so pushes to main are never cancelled by the next merge - drop -DSVS_EXPERIMENTAL_LEANVEC=YES (no such CMake option) - cibuildwheel: pull_request paths filter - core builds: ignore CODEOWNERS, dependabot, unrelated workflows, LICENSE, THIRD-PARTY-PROGRAMS --- .github/workflows/build-c-api-bindings.yml | 8 +++++++- .github/workflows/build-cpp-runtime-bindings.yml | 8 +++++++- .github/workflows/build-linux-arm.yml | 11 ++++++++--- .github/workflows/build-linux.yml | 8 +++++++- .github/workflows/build-macos.yaml | 11 ++++++++--- .github/workflows/cibuildwheel.yml | 12 +++++++++++- .github/workflows/skywalking-eyes.yml | 2 +- 7 files changed, 49 insertions(+), 11 deletions(-) diff --git a/.github/workflows/build-c-api-bindings.yml b/.github/workflows/build-c-api-bindings.yml index bce1fcfa2..3a6d3566b 100644 --- a/.github/workflows/build-c-api-bindings.yml +++ b/.github/workflows/build-c-api-bindings.yml @@ -23,6 +23,12 @@ on: - 'bindings/cpp/**' - 'bindings/python/**' - '**.md' + - '.github/CODEOWNERS' + - '.github/dependabot.yml' + - '.github/workflows/cibuildwheel.yml' + - '.github/workflows/skywalking-eyes.yml' + - 'LICENSE*' + - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: @@ -31,7 +37,7 @@ permissions: # This allows a subsequently queued workflow run to interrupt previous runs concurrency: group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}' - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: build-c-api-bindings: diff --git a/.github/workflows/build-cpp-runtime-bindings.yml b/.github/workflows/build-cpp-runtime-bindings.yml index c18e1a8c6..f4ae5d16c 100644 --- a/.github/workflows/build-cpp-runtime-bindings.yml +++ b/.github/workflows/build-cpp-runtime-bindings.yml @@ -23,6 +23,12 @@ on: - 'bindings/c/**' - 'bindings/python/**' - '**.md' + - '.github/CODEOWNERS' + - '.github/dependabot.yml' + - '.github/workflows/cibuildwheel.yml' + - '.github/workflows/skywalking-eyes.yml' + - 'LICENSE*' + - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: @@ -31,7 +37,7 @@ permissions: # This allows a subsequently queued workflow run to interrupt previous runs concurrency: group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}' - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: build-cpp-runtime-bindings: diff --git a/.github/workflows/build-linux-arm.yml b/.github/workflows/build-linux-arm.yml index 5a46f0361..d4fec6765 100644 --- a/.github/workflows/build-linux-arm.yml +++ b/.github/workflows/build-linux-arm.yml @@ -22,6 +22,12 @@ on: paths-ignore: - 'bindings/**' - '**.md' + - '.github/CODEOWNERS' + - '.github/dependabot.yml' + - '.github/workflows/cibuildwheel.yml' + - '.github/workflows/skywalking-eyes.yml' + - 'LICENSE*' + - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: @@ -30,7 +36,7 @@ permissions: # This allows a subsequently queued workflow run to interrupt previous runs concurrency: group: ${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }} - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: build: @@ -62,8 +68,7 @@ jobs: -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} \ -DSVS_BUILD_BINARIES=YES \ -DSVS_BUILD_TESTS=YES \ - -DSVS_BUILD_EXAMPLES=YES \ - -DSVS_EXPERIMENTAL_LEANVEC=YES + -DSVS_BUILD_EXAMPLES=YES - name: Build Tests and Utilities working-directory: ${{ runner.temp }}/build diff --git a/.github/workflows/build-linux.yml b/.github/workflows/build-linux.yml index 92df99a86..8322d80bb 100644 --- a/.github/workflows/build-linux.yml +++ b/.github/workflows/build-linux.yml @@ -22,6 +22,12 @@ on: paths-ignore: - 'bindings/**' - '**.md' + - '.github/CODEOWNERS' + - '.github/dependabot.yml' + - '.github/workflows/cibuildwheel.yml' + - '.github/workflows/skywalking-eyes.yml' + - 'LICENSE*' + - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: @@ -30,7 +36,7 @@ permissions: # This allows a subsequently queued workflow run to interrupt previous runs concurrency: group: ${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }} - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: build: diff --git a/.github/workflows/build-macos.yaml b/.github/workflows/build-macos.yaml index 47649c0a4..1927a1750 100644 --- a/.github/workflows/build-macos.yaml +++ b/.github/workflows/build-macos.yaml @@ -22,6 +22,12 @@ on: paths-ignore: - 'bindings/**' - '**.md' + - '.github/CODEOWNERS' + - '.github/dependabot.yml' + - '.github/workflows/cibuildwheel.yml' + - '.github/workflows/skywalking-eyes.yml' + - 'LICENSE*' + - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: @@ -30,7 +36,7 @@ permissions: # This allows a subsequently queued workflow run to interrupt previous runs concurrency: group: ${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }} - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: build: @@ -75,8 +81,7 @@ jobs: -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} \ -DSVS_BUILD_BINARIES=YES \ -DSVS_BUILD_TESTS=YES \ - -DSVS_BUILD_EXAMPLES=YES \ - -DSVS_EXPERIMENTAL_LEANVEC=YES + -DSVS_BUILD_EXAMPLES=YES - name: Build Tests and Utilities working-directory: ${{ runner.temp }}/build diff --git a/.github/workflows/cibuildwheel.yml b/.github/workflows/cibuildwheel.yml index 6ab5644f9..f688a6e09 100644 --- a/.github/workflows/cibuildwheel.yml +++ b/.github/workflows/cibuildwheel.yml @@ -19,6 +19,16 @@ on: branches: - main pull_request: + paths: + - 'bindings/python/**' + - 'include/**' + - 'cmake/**' + - 'CMakeLists.txt' + - 'data/**' + - 'docker/x86_64/manylinux228/**' + - 'examples/python/**' + - 'tools/pybuild.py' + - '.github/workflows/cibuildwheel.yml' workflow_dispatch: permissions: @@ -26,7 +36,7 @@ permissions: concurrency: group: ${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }} - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: python-build: diff --git a/.github/workflows/skywalking-eyes.yml b/.github/workflows/skywalking-eyes.yml index aed657dfe..9ded33bd0 100644 --- a/.github/workflows/skywalking-eyes.yml +++ b/.github/workflows/skywalking-eyes.yml @@ -27,7 +27,7 @@ permissions: read-all concurrency: group: ${{ github.workflow }}-${{ github.event_name }}-${{ github.ref_name }}-${{ github.event.number || github.sha }} - cancel-in-progress: true + cancel-in-progress: ${{ github.event_name == 'pull_request' }} jobs: check-license-header: From 94020726ecbb39be3e66563c1223347845ed64dd Mon Sep 17 00:00:00 2001 From: Nikolay Petrov Date: Sun, 25 Oct 2026 22:33:05 -0700 Subject: [PATCH 2/2] ci: trigger wheel and runtime builds on the root files they read setup.py reads README.md, LICENSE and THIRD-PARTY-PROGRAMS from the repo root, and the cpp runtime conda recipe packages LICENSE and THIRD-PARTY-PROGRAMS. The new path filter and paths-ignore skipped those changes, so a rename or removal would only fail after merge. --- .github/workflows/build-cpp-runtime-bindings.yml | 2 -- .github/workflows/cibuildwheel.yml | 4 ++++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build-cpp-runtime-bindings.yml b/.github/workflows/build-cpp-runtime-bindings.yml index f4ae5d16c..2205737ef 100644 --- a/.github/workflows/build-cpp-runtime-bindings.yml +++ b/.github/workflows/build-cpp-runtime-bindings.yml @@ -27,8 +27,6 @@ on: - '.github/dependabot.yml' - '.github/workflows/cibuildwheel.yml' - '.github/workflows/skywalking-eyes.yml' - - 'LICENSE*' - - 'THIRD-PARTY-PROGRAMS' workflow_dispatch: permissions: diff --git a/.github/workflows/cibuildwheel.yml b/.github/workflows/cibuildwheel.yml index f688a6e09..2580ace89 100644 --- a/.github/workflows/cibuildwheel.yml +++ b/.github/workflows/cibuildwheel.yml @@ -28,6 +28,10 @@ on: - 'docker/x86_64/manylinux228/**' - 'examples/python/**' - 'tools/pybuild.py' + # setup.py reads these from the repo root (long description, license_files) + - 'README.md' + - 'LICENSE' + - 'THIRD-PARTY-PROGRAMS' - '.github/workflows/cibuildwheel.yml' workflow_dispatch: