From ce7d8e6b0295c4feba81e7d0b0cc16da4c1f4b03 Mon Sep 17 00:00:00 2001 From: Pankaj Koti Date: Fri, 27 Mar 2026 00:08:41 +0530 Subject: [PATCH 1/2] Skip tests in CI for non-code file changes Expand the Check-changed-files skip list beyond docs to also cover metadata files (LICENSE, SECURITY.rst, CODEOWNERS, etc.), GitHub/CI config unrelated to tests (issue templates, dependabot, non-test workflows), and tooling config (.pre-commit-config.yaml, codecov.yml, Tiltfile, etc.). Pre-commit and type-check jobs still run unconditionally. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/test.yml | 30 +++++++++++++++++++++++++++--- 1 file changed, 27 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 828102f4eb..d1ff1bcbd3 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -27,7 +27,7 @@ jobs: steps: - run: true - # Skip unit/integration tests when only documentation changed (pre-commit/Type-Check still run). + # Skip unit/integration tests when only non-code files changed (pre-commit/Type-Check still run). Check-changed-files: needs: Authorize runs-on: ubuntu-latest @@ -69,12 +69,36 @@ jobs: while IFS= read -r f; do [ -z "$f" ] && continue case "$f" in - README.md) ;; + # Documentation and metadata + README.rst|README.md) ;; CHANGELOG.rst) ;; CODE_OF_CONDUCT.md) ;; CONTRIBUTING.md) ;; - .github/pull_request_template.md) ;; + LICENSE) ;; + SECURITY.rst) ;; + PRIVACY_NOTICE.rst) ;; + CODEOWNERS) ;; + CLAUDE.md) ;; docs/*) ;; + # GitHub and CI config unrelated to tests + .github/pull_request_template.md) ;; + .github/ISSUE_TEMPLATE/*) ;; + .github/dependabot.yml) ;; + .github/workflows/docs.yml) ;; + .github/workflows/stale.yml) ;; + .github/workflows/actionlint.yml) ;; + .github/workflows/codeql.yml) ;; + .github/workflows/zizmor.yml) ;; + .github/workflows/deploy.yml) ;; + # Tooling config that does not affect test outcomes + .gitignore) ;; + .tiltignore) ;; + Tiltfile) ;; + .codespell-ignore-words) ;; + .pre-commit-config.yaml) ;; + codecov.yml) ;; + .airflow-registry.yaml) ;; + .coveragerc) ;; *) CODE_CHANGED=true; break ;; esac done <<< "$FILES" From 51cb49335abf038ff569fb8aaae41fd3ca8b380e Mon Sep 17 00:00:00 2001 From: Pankaj Koti Date: Fri, 27 Mar 2026 00:43:51 +0530 Subject: [PATCH 2/2] Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/workflows/test.yml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index d1ff1bcbd3..ce456f20e9 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -27,7 +27,7 @@ jobs: steps: - run: true - # Skip unit/integration tests when only non-code files changed (pre-commit/Type-Check still run). + # Skip unit/integration tests when only non-code files changed (Type-Check still runs). Check-changed-files: needs: Authorize runs-on: ubuntu-latest @@ -96,9 +96,7 @@ jobs: Tiltfile) ;; .codespell-ignore-words) ;; .pre-commit-config.yaml) ;; - codecov.yml) ;; .airflow-registry.yaml) ;; - .coveragerc) ;; *) CODE_CHANGED=true; break ;; esac done <<< "$FILES"