From 268e6f3a0eaa1d1f240763ef44b0e88eead632ef Mon Sep 17 00:00:00 2001 From: Matjaz Pirnovar Date: Thu, 23 Apr 2026 10:56:12 -0700 Subject: [PATCH 1/4] [FSSDK-12539] Remove Coveralls from CI checks Remove Coveralls coverage upload and README badge. Tests still run via make cover but no longer report to Coveralls. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/agent.yml | 12 ++---------- README.md | 1 - 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/.github/workflows/agent.yml b/.github/workflows/agent.yml index 8fe08039..45175246 100644 --- a/.github/workflows/agent.yml +++ b/.github/workflows/agent.yml @@ -59,10 +59,8 @@ jobs: ignore: "DL3018" dockerfile: scripts/dockerfiles/Dockerfile.* - tests_coveralls: + tests: runs-on: ubuntu-latest - env: - COVERALLS_TOKEN: ${{ secrets.GITHUB_TOKEN }} steps: - uses: actions/checkout@v3 with: @@ -75,16 +73,10 @@ jobs: uses: supercharge/redis-github-action@1.5.0 with: redis-version: 5 - - name: coveralls - id: coveralls + - name: test run: | make -e setup build make -e cover COVER_FILE=coverage.txt - - name: success - if: steps.coveralls.outcome == 'success' - run: | - go install github.com/mattn/goveralls@latest - goveralls -coverprofile=coverage.txt -service=github tests_windows_build_ps1: runs-on: windows-2022 diff --git a/README.md b/README.md index 6d8fe555..8aededbb 100644 --- a/README.md +++ b/README.md @@ -1,5 +1,4 @@ [![Build Status](https://github.com/optimizely/agent/actions/workflows/agent.yml/badge.svg?branch=master)](https://github.com/optimizely/agent/actions/workflows/agent.yml?query=branch%3Amaster) -[![Coverage Status](https://coveralls.io/repos/github/optimizely/agent/badge.svg)](https://coveralls.io/github/optimizely/agent) # Optimizely Agent From 22ca9d6242870c257a0b6034ba4167008eb3195b Mon Sep 17 00:00:00 2001 From: Matjaz Pirnovar Date: Thu, 23 Apr 2026 11:06:11 -0700 Subject: [PATCH 2/4] [FSSDK-12539] Restore tests_coveralls job name for branch protection Keep the job name so required check still passes, but remove the goveralls upload step. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/agent.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/agent.yml b/.github/workflows/agent.yml index 45175246..935f456f 100644 --- a/.github/workflows/agent.yml +++ b/.github/workflows/agent.yml @@ -59,7 +59,7 @@ jobs: ignore: "DL3018" dockerfile: scripts/dockerfiles/Dockerfile.* - tests: + tests_coveralls: runs-on: ubuntu-latest steps: - uses: actions/checkout@v3 From 7853162f2c329445c69bc711da7772a8d4f9ce35 Mon Sep 17 00:00:00 2001 From: Matjaz Pirnovar Date: Thu, 23 Apr 2026 11:49:30 -0700 Subject: [PATCH 3/4] Rename tests_coveralls job to unit_test_coverage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Addresses reviewer feedback — Coveralls is removed, so the job name should reflect that. Aligns with go-sdk naming convention. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/agent.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/agent.yml b/.github/workflows/agent.yml index 935f456f..8d761f8f 100644 --- a/.github/workflows/agent.yml +++ b/.github/workflows/agent.yml @@ -59,7 +59,7 @@ jobs: ignore: "DL3018" dockerfile: scripts/dockerfiles/Dockerfile.* - tests_coveralls: + unit_test_coverage: runs-on: ubuntu-latest steps: - uses: actions/checkout@v3 From 58ec88c4c1fc535e2bde999c547d63aecd42dcfe Mon Sep 17 00:00:00 2001 From: Matjaz Pirnovar Date: Thu, 23 Apr 2026 12:00:24 -0700 Subject: [PATCH 4/4] Trigger CI checks