From 626a80fe1ac7d86cdc1aec25c51bba7158b98a44 Mon Sep 17 00:00:00 2001 From: manuzhang Date: Wed, 22 Jul 2026 17:09:02 +0800 Subject: [PATCH 1/2] ci: scope pull request checks to relevant paths Filter pull request checks without duplicating path lists on push triggers. Run Actions validators for GitHub YAML changes, and skip build, test, and lint workflows for documentation and development-only paths. Keep license and pre-commit checks unfiltered. Co-authored-by: Codex --- .github/workflows/asf-allowlist-check.yml | 3 +++ .github/workflows/aws_test.yml | 6 ++++++ .github/workflows/codeql.yml | 3 +++ .github/workflows/cpp-linter.yml | 9 +++++---- .github/workflows/sanitizer_test.yml | 6 ++++++ .github/workflows/sql_catalog_test.yml | 6 ++++++ .github/workflows/test.yml | 6 ++++++ .github/workflows/zizmor.yml | 3 +++ 8 files changed, 38 insertions(+), 4 deletions(-) diff --git a/.github/workflows/asf-allowlist-check.yml b/.github/workflows/asf-allowlist-check.yml index d642d5c59..65d241aff 100644 --- a/.github/workflows/asf-allowlist-check.yml +++ b/.github/workflows/asf-allowlist-check.yml @@ -29,6 +29,9 @@ on: - main pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths: + - '.github/**/*.yml' + - '.github/**/*.yaml' permissions: contents: read diff --git a/.github/workflows/aws_test.yml b/.github/workflows/aws_test.yml index 0ed9685b7..6d0e3ae97 100644 --- a/.github/workflows/aws_test.yml +++ b/.github/workflows/aws_test.yml @@ -25,6 +25,12 @@ on: - '**' pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths-ignore: + - '.devcontainer/**' + - '*.md' + - '**/*.md' + - 'dev/**' + - 'mkdocs/**' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml index 1dc7ee053..b4780aa9b 100644 --- a/.github/workflows/codeql.yml +++ b/.github/workflows/codeql.yml @@ -27,6 +27,9 @@ on: types: [opened, synchronize, reopened, ready_for_review] branches: - main + paths: + - '.github/**/*.yml' + - '.github/**/*.yaml' schedule: - cron: '16 4 * * 1' diff --git a/.github/workflows/cpp-linter.yml b/.github/workflows/cpp-linter.yml index 44dd6e350..62f28e286 100644 --- a/.github/workflows/cpp-linter.yml +++ b/.github/workflows/cpp-linter.yml @@ -21,18 +21,19 @@ on: push: branches: - main - paths-ignore: - - '.github/**' - - 'ci/**' - - 'cmake_modules/**' pull_request: types: [opened, synchronize, reopened, ready_for_review] branches: - main paths-ignore: + - '.devcontainer/**' - '.github/**' + - '*.md' + - '**/*.md' - 'ci/**' - 'cmake_modules/**' + - 'dev/**' + - 'mkdocs/**' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} diff --git a/.github/workflows/sanitizer_test.yml b/.github/workflows/sanitizer_test.yml index cd9797b9f..cc521837f 100644 --- a/.github/workflows/sanitizer_test.yml +++ b/.github/workflows/sanitizer_test.yml @@ -25,6 +25,12 @@ on: - '**' pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths-ignore: + - '.devcontainer/**' + - '*.md' + - '**/*.md' + - 'dev/**' + - 'mkdocs/**' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} diff --git a/.github/workflows/sql_catalog_test.yml b/.github/workflows/sql_catalog_test.yml index 108ff8ae2..eb00a01bc 100644 --- a/.github/workflows/sql_catalog_test.yml +++ b/.github/workflows/sql_catalog_test.yml @@ -25,6 +25,12 @@ on: - '**' pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths-ignore: + - '.devcontainer/**' + - '*.md' + - '**/*.md' + - 'dev/**' + - 'mkdocs/**' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 4901e8ea9..291b73e80 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -25,6 +25,12 @@ on: - '**' pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths-ignore: + - '.devcontainer/**' + - '*.md' + - '**/*.md' + - 'dev/**' + - 'mkdocs/**' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} diff --git a/.github/workflows/zizmor.yml b/.github/workflows/zizmor.yml index 820196470..e51e69d3a 100644 --- a/.github/workflows/zizmor.yml +++ b/.github/workflows/zizmor.yml @@ -25,6 +25,9 @@ on: - main pull_request: types: [opened, synchronize, reopened, ready_for_review] + paths: + - '.github/**/*.yml' + - '.github/**/*.yaml' concurrency: group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }} From f322ce97f2e05783a7af4c6989e43986e6f98467 Mon Sep 17 00:00:00 2001 From: manuzhang Date: Wed, 22 Jul 2026 17:57:11 +0800 Subject: [PATCH 2/2] ci: remove redundant markdown path globs Co-authored-by: Codex --- .github/workflows/aws_test.yml | 1 - .github/workflows/cpp-linter.yml | 1 - .github/workflows/sanitizer_test.yml | 1 - .github/workflows/sql_catalog_test.yml | 1 - .github/workflows/test.yml | 1 - 5 files changed, 5 deletions(-) diff --git a/.github/workflows/aws_test.yml b/.github/workflows/aws_test.yml index 6d0e3ae97..89d4c22b8 100644 --- a/.github/workflows/aws_test.yml +++ b/.github/workflows/aws_test.yml @@ -27,7 +27,6 @@ on: types: [opened, synchronize, reopened, ready_for_review] paths-ignore: - '.devcontainer/**' - - '*.md' - '**/*.md' - 'dev/**' - 'mkdocs/**' diff --git a/.github/workflows/cpp-linter.yml b/.github/workflows/cpp-linter.yml index 62f28e286..8dc225050 100644 --- a/.github/workflows/cpp-linter.yml +++ b/.github/workflows/cpp-linter.yml @@ -28,7 +28,6 @@ on: paths-ignore: - '.devcontainer/**' - '.github/**' - - '*.md' - '**/*.md' - 'ci/**' - 'cmake_modules/**' diff --git a/.github/workflows/sanitizer_test.yml b/.github/workflows/sanitizer_test.yml index cc521837f..1a4c7bdb2 100644 --- a/.github/workflows/sanitizer_test.yml +++ b/.github/workflows/sanitizer_test.yml @@ -27,7 +27,6 @@ on: types: [opened, synchronize, reopened, ready_for_review] paths-ignore: - '.devcontainer/**' - - '*.md' - '**/*.md' - 'dev/**' - 'mkdocs/**' diff --git a/.github/workflows/sql_catalog_test.yml b/.github/workflows/sql_catalog_test.yml index eb00a01bc..9bd4be516 100644 --- a/.github/workflows/sql_catalog_test.yml +++ b/.github/workflows/sql_catalog_test.yml @@ -27,7 +27,6 @@ on: types: [opened, synchronize, reopened, ready_for_review] paths-ignore: - '.devcontainer/**' - - '*.md' - '**/*.md' - 'dev/**' - 'mkdocs/**' diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 291b73e80..7b21bcf5f 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -27,7 +27,6 @@ on: types: [opened, synchronize, reopened, ready_for_review] paths-ignore: - '.devcontainer/**' - - '*.md' - '**/*.md' - 'dev/**' - 'mkdocs/**'