From 2978297d702f63aec6119e37fdc3ac43052046ff Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dean=20Qui=C3=B1anola?= Date: Tue, 24 Feb 2026 16:40:23 -0800 Subject: [PATCH 1/2] fix(ci): eliminate redundant CI runs on release-please PRs and main pushes - Skip test, lint, and Docker build jobs on release-please PRs (metadata-only changes) - Remove duplicate test-unit/test-integration steps on main (test-coverage already runs full suite) - Migrate from deprecated google-github-actions/release-please-action to googleapis/release-please-action - Update docker-validation to accept skipped upstream jobs gracefully --- .github/workflows/ci.yml | 30 +++++++++++++----------------- 1 file changed, 13 insertions(+), 17 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d61c13d..3693684 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -19,6 +19,7 @@ env: jobs: test: runs-on: ubuntu-latest + if: github.event_name != 'pull_request' || github.head_ref != 'release-please--branches--main' strategy: matrix: python-version: ${{ github.event_name == 'pull_request' && fromJSON('["3.11", "3.12"]') || fromJSON('["3.10", "3.11", "3.12", "3.13", "3.14"]') }} @@ -48,16 +49,9 @@ jobs: - name: Run all tests with coverage run: make test-coverage - - name: Run unit tests separately (main branch only) - if: github.ref == 'refs/heads/main' - run: make test-unit - - - name: Run integration tests separately (main branch only) - if: github.ref == 'refs/heads/main' - run: make test-integration - lint: runs-on: ubuntu-latest + if: github.event_name != 'pull_request' || github.head_ref != 'release-please--branches--main' steps: - name: Checkout repository uses: actions/checkout@v4 @@ -80,6 +74,7 @@ jobs: docker-test: runs-on: ubuntu-latest + if: github.event_name != 'pull_request' || github.head_ref != 'release-please--branches--main' needs: [lint] steps: - name: Clear Space @@ -131,6 +126,7 @@ jobs: docker-test-lb: runs-on: ubuntu-latest + if: github.event_name != 'pull_request' || github.head_ref != 'release-please--branches--main' needs: [lint] steps: - name: Free Disk Space @@ -181,6 +177,7 @@ jobs: docker-test-lb-cpu: runs-on: ubuntu-latest + if: github.event_name != 'pull_request' || github.head_ref != 'release-please--branches--main' needs: [lint] steps: - name: Clear Space @@ -232,14 +229,13 @@ jobs: steps: - name: Check all jobs succeeded run: | - if [[ "${{ needs.test.result }}" != "success" ]] || \ - [[ "${{ needs.lint.result }}" != "success" ]] || \ - [[ "${{ needs.docker-test.result }}" != "success" ]] || \ - [[ "${{ needs.docker-test-lb.result }}" != "success" ]] || \ - [[ "${{ needs.docker-test-lb-cpu.result }}" != "success" ]]; then - echo "One or more quality checks failed" - exit 1 - fi + results=("${{ needs.test.result }}" "${{ needs.lint.result }}" "${{ needs.docker-test.result }}" "${{ needs.docker-test-lb.result }}" "${{ needs.docker-test-lb-cpu.result }}") + for result in "${results[@]}"; do + if [[ "$result" != "success" && "$result" != "skipped" ]]; then + echo "One or more quality checks failed (got: $result)" + exit 1 + fi + done release: runs-on: ubuntu-latest @@ -256,7 +252,7 @@ jobs: app-id: ${{ secrets.RELEASE_APP_ID }} private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }} - - uses: google-github-actions/release-please-action@v4 + - uses: googleapis/release-please-action@v4 id: release with: token: ${{ steps.app-token.outputs.token }} From 522cbb498f5e912fa029d3e777d204fe0fdc4c64 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dean=20Qui=C3=B1anola?= Date: Tue, 24 Feb 2026 21:49:56 -0800 Subject: [PATCH 2/2] chore: sync uv.lock version to 1.1.0 --- uv.lock | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/uv.lock b/uv.lock index 8ce206c..42d4810 100644 --- a/uv.lock +++ b/uv.lock @@ -3701,7 +3701,7 @@ wheels = [ [[package]] name = "worker-flash" -version = "1.0.1" +version = "1.1.0" source = { virtual = "." } dependencies = [ { name = "aiohttp" },