From ce69a3571efbb47fb629e75ef50a6f7153d190e8 Mon Sep 17 00:00:00 2001 From: Ivan Shymko Date: Mon, 20 Apr 2026 14:41:31 +0000 Subject: [PATCH 1/4] ci: add some path filters Do not run heavy jobs for docs and other unrelated changes. --- .github/workflows/linter.yaml | 11 +++++++++++ .github/workflows/minimal-install.yml | 22 ++++++++++++++++++++++ .github/workflows/run-tck.yaml | 18 ++++++++++++++++++ .github/workflows/unit-tests.yml | 22 ++++++++++++++++++++++ 4 files changed, 73 insertions(+) diff --git a/.github/workflows/linter.yaml b/.github/workflows/linter.yaml index 4263abb3c..ec4bd16fb 100644 --- a/.github/workflows/linter.yaml +++ b/.github/workflows/linter.yaml @@ -3,6 +3,17 @@ name: Lint Code Base on: pull_request: branches: [main, 1.0-dev] + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' permissions: contents: read jobs: diff --git a/.github/workflows/minimal-install.yml b/.github/workflows/minimal-install.yml index 7e0f143c6..27afebe7e 100644 --- a/.github/workflows/minimal-install.yml +++ b/.github/workflows/minimal-install.yml @@ -3,7 +3,29 @@ name: Minimal Install Smoke Test on: push: branches: [main, 1.0-dev] + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' pull_request: + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' permissions: contents: read diff --git a/.github/workflows/run-tck.yaml b/.github/workflows/run-tck.yaml index 6d0df865f..62bbeebc0 100644 --- a/.github/workflows/run-tck.yaml +++ b/.github/workflows/run-tck.yaml @@ -3,12 +3,30 @@ name: Run TCK on: push: branches: [ "main" ] + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' pull_request: branches: [ "main" ] paths-ignore: - '**.md' - 'LICENSE' + - 'docs/**' - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' permissions: contents: read diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml index adabe0676..098a14ecc 100644 --- a/.github/workflows/unit-tests.yml +++ b/.github/workflows/unit-tests.yml @@ -3,7 +3,29 @@ name: Run Unit Tests on: push: branches: [main, 1.0-dev] + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' pull_request: + paths-ignore: + - '**.md' + - 'LICENSE' + - 'docs/**' + - '.github/CODEOWNERS' + - '.github/ISSUE_TEMPLATE/**' + - '.github/PULL_REQUEST_TEMPLATE.md' + - '.github/dependabot.yml' + - '.gitignore' + - '.git-blame-ignore-revs' + - '.gemini/**' permissions: contents: read From 4eb643217e0851a49469a8f09ce93e0df6e13ac3 Mon Sep 17 00:00:00 2001 From: Ivan Shymko Date: Mon, 20 Apr 2026 14:49:37 +0000 Subject: [PATCH 2/4] WIP --- .github/workflows/itk.yaml | 8 ++++---- .github/workflows/minimal-install.yml | 11 ----------- .github/workflows/run-tck.yaml | 11 ----------- .github/workflows/unit-tests.yml | 11 ----------- 4 files changed, 4 insertions(+), 37 deletions(-) diff --git a/.github/workflows/itk.yaml b/.github/workflows/itk.yaml index ab272d0e3..987456264 100644 --- a/.github/workflows/itk.yaml +++ b/.github/workflows/itk.yaml @@ -2,12 +2,12 @@ name: ITK on: push: - branches: [main, 1.0-dev] + branches: [ main, 1.0-dev ] pull_request: paths: - - 'src/**' - - 'itk/**' - - 'pyproject.toml' + - "src/**" + - "itk/**" + - "pyproject.toml" permissions: contents: read diff --git a/.github/workflows/minimal-install.yml b/.github/workflows/minimal-install.yml index 27afebe7e..70ad6ba5e 100644 --- a/.github/workflows/minimal-install.yml +++ b/.github/workflows/minimal-install.yml @@ -3,17 +3,6 @@ name: Minimal Install Smoke Test on: push: branches: [main, 1.0-dev] - paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' pull_request: paths-ignore: - '**.md' diff --git a/.github/workflows/run-tck.yaml b/.github/workflows/run-tck.yaml index 62bbeebc0..72e598007 100644 --- a/.github/workflows/run-tck.yaml +++ b/.github/workflows/run-tck.yaml @@ -3,17 +3,6 @@ name: Run TCK on: push: branches: [ "main" ] - paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' pull_request: branches: [ "main" ] paths-ignore: diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml index 098a14ecc..a16961e66 100644 --- a/.github/workflows/unit-tests.yml +++ b/.github/workflows/unit-tests.yml @@ -3,17 +3,6 @@ name: Run Unit Tests on: push: branches: [main, 1.0-dev] - paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' pull_request: paths-ignore: - '**.md' From 8130aa2f3be84982be62a92bf33a1eff84078f30 Mon Sep 17 00:00:00 2001 From: Ivan Shymko Date: Mon, 20 Apr 2026 15:13:31 +0000 Subject: [PATCH 3/4] ci: cleanup 1.0-dev triggers and remove branch filters 1. Cleanup 1.0-dev filters as the branch won't be used going forward. 2. Remove any branch filters for workflows as a safety net if something got skipped on PR by accident. --- .github/workflows/itk.yaml | 2 +- .github/workflows/minimal-install.yml | 24 +++++++++---------- .github/workflows/unit-tests.yml | 33 ++++++++++++++------------- 3 files changed, 30 insertions(+), 29 deletions(-) diff --git a/.github/workflows/itk.yaml b/.github/workflows/itk.yaml index 987456264..4802c6186 100644 --- a/.github/workflows/itk.yaml +++ b/.github/workflows/itk.yaml @@ -2,7 +2,7 @@ name: ITK on: push: - branches: [ main, 1.0-dev ] + branches: [ main ] pull_request: paths: - "src/**" diff --git a/.github/workflows/minimal-install.yml b/.github/workflows/minimal-install.yml index 70ad6ba5e..91987873d 100644 --- a/.github/workflows/minimal-install.yml +++ b/.github/workflows/minimal-install.yml @@ -2,19 +2,19 @@ name: Minimal Install Smoke Test on: push: - branches: [main, 1.0-dev] + branches: [ main ] pull_request: paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' + - "**.md" + - "LICENSE" + - "docs/**" + - ".github/CODEOWNERS" + - ".github/ISSUE_TEMPLATE/**" + - ".github/PULL_REQUEST_TEMPLATE.md" + - ".github/dependabot.yml" + - ".gitignore" + - ".git-blame-ignore-revs" + - ".gemini/**" permissions: contents: read @@ -25,7 +25,7 @@ jobs: if: github.repository == 'a2aproject/a2a-python' strategy: matrix: - python-version: ['3.10', '3.11', '3.12', '3.13', '3.14'] + python-version: [ "3.10", "3.11", "3.12", "3.13", "3.14" ] steps: - name: Checkout code uses: actions/checkout@v6 diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml index a16961e66..90d6f0102 100644 --- a/.github/workflows/unit-tests.yml +++ b/.github/workflows/unit-tests.yml @@ -2,19 +2,19 @@ name: Run Unit Tests on: push: - branches: [main, 1.0-dev] + branches: [ main ] pull_request: paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' + - "**.md" + - "LICENSE" + - "docs/**" + - ".github/CODEOWNERS" + - ".github/ISSUE_TEMPLATE/**" + - ".github/PULL_REQUEST_TEMPLATE.md" + - ".github/dependabot.yml" + - ".gitignore" + - ".git-blame-ignore-revs" + - ".gemini/**" permissions: contents: read @@ -34,7 +34,8 @@ jobs: ports: - 5432:5432 options: >- - --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5 + --health-cmd pg_isready --health-interval 10s --health-timeout 5s + --health-retries 5 mysql: image: mysql:8.0 env: @@ -45,11 +46,12 @@ jobs: ports: - 3306:3306 options: >- - --health-cmd="mysqladmin ping -h localhost -u root -proot" --health-interval=10s --health-timeout=5s --health-retries=5 + --health-cmd="mysqladmin ping -h localhost -u root -proot" + --health-interval=10s --health-timeout=5s --health-retries=5 strategy: matrix: - python-version: ['3.10', '3.11', '3.12', '3.13', '3.14'] + python-version: [ "3.10", "3.11", "3.12", "3.13", "3.14" ] steps: - name: Checkout code uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6 @@ -66,7 +68,6 @@ jobs: run: | echo "$HOME/.cargo/bin" >> $GITHUB_PATH - # Coverage comparison for PRs (only on Python 3.14 to avoid duplicate work) - name: Checkout Base Branch if: github.event_name == 'pull_request' && matrix.python-version == '3.14' @@ -74,7 +75,7 @@ jobs: with: ref: ${{ github.event.pull_request.base.ref || 'main' }} clean: true - + - name: Install dependencies run: uv sync --locked From 30dd0a402e7a0237359f4a34c0cc952e8ef13524 Mon Sep 17 00:00:00 2001 From: Ivan Shymko Date: Mon, 20 Apr 2026 15:18:00 +0000 Subject: [PATCH 4/4] Update linter.yaml --- .github/workflows/linter.yaml | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/.github/workflows/linter.yaml b/.github/workflows/linter.yaml index ec4bd16fb..ebc5255dc 100644 --- a/.github/workflows/linter.yaml +++ b/.github/workflows/linter.yaml @@ -2,18 +2,18 @@ name: Lint Code Base on: pull_request: - branches: [main, 1.0-dev] + branches: [ main ] paths-ignore: - - '**.md' - - 'LICENSE' - - 'docs/**' - - '.github/CODEOWNERS' - - '.github/ISSUE_TEMPLATE/**' - - '.github/PULL_REQUEST_TEMPLATE.md' - - '.github/dependabot.yml' - - '.gitignore' - - '.git-blame-ignore-revs' - - '.gemini/**' + - "**.md" + - "LICENSE" + - "docs/**" + - ".github/CODEOWNERS" + - ".github/ISSUE_TEMPLATE/**" + - ".github/PULL_REQUEST_TEMPLATE.md" + - ".github/dependabot.yml" + - ".gitignore" + - ".git-blame-ignore-revs" + - ".gemini/**" permissions: contents: read jobs: