From 3b30f1e83eee824c8d354e681103c5346267011d Mon Sep 17 00:00:00 2001 From: dkijania Date: Tue, 14 Apr 2026 11:33:30 +0200 Subject: [PATCH 1/2] Add Codecov badge to README Co-Authored-By: Claude Opus 4.6 (1M context) --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index 9f2472b..7d42070 100644 --- a/README.md +++ b/README.md @@ -2,6 +2,7 @@ [![CI](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/ci.yml/badge.svg)](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/ci.yml) [![Integration Tests](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/integration.yml/badge.svg)](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/integration.yml) +[![codecov](https://codecov.io/gh/MinaProtocol/mina-sdk-rust/graph/badge.svg)](https://codecov.io/gh/MinaProtocol/mina-sdk-rust) [![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](LICENSE) Rust SDK for interacting with [Mina Protocol](https://minaprotocol.com) nodes via GraphQL. From f2b9553b807ce729d29565376cb817ded1ba145a Mon Sep 17 00:00:00 2001 From: dkijania Date: Sat, 18 Apr 2026 00:28:43 +0200 Subject: [PATCH 2/2] Replace Codecov with Coveralls and cover the client MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Use Coveralls instead of Codecov: the previous setup uploaded to Codecov without a token, got rejected, and silently passed CI (fail_ci_if_error was false) — so the dashboard and badge were empty. Coveralls picks up GITHUB_TOKEN automatically for public repos, no secret to configure. - Run client_tests.rs alongside the lib unit tests so src/client.rs is actually exercised (raised coverage from 18% to 88% locally). Keep integration_tests.rs out of the coverage job since it needs a live daemon. - Exclude examples/ and tests/ from the coverage report. - Switch the README badge to Coveralls. - .gitignore the generated tarpaulin artifacts. Co-Authored-By: Claude Opus 4.7 (1M context) --- .github/workflows/ci.yml | 17 ++++++++++------- .gitignore | 3 +++ README.md | 2 +- 3 files changed, 14 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a2cab09..056f475 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -59,11 +59,14 @@ jobs: run: cargo install cargo-tarpaulin - name: Generate coverage - run: cargo tarpaulin --lib --out xml --skip-clean -- --skip integration - - - name: Upload to Codecov - if: github.event_name != 'pull_request' - uses: codecov/codecov-action@v6 + run: > + cargo tarpaulin --lib --test client_tests + --out lcov --skip-clean + --exclude-files 'examples/*' + --exclude-files 'tests/*' + + - name: Upload to Coveralls + uses: coverallsapp/github-action@v2 with: - files: cobertura.xml - fail_ci_if_error: false + file: lcov.info + fail-on-error: true diff --git a/.gitignore b/.gitignore index ba1e343..adf4515 100644 --- a/.gitignore +++ b/.gitignore @@ -5,3 +5,6 @@ Cargo.lock *.swp *.swo *~ +cobertura.xml +lcov.info +tarpaulin-report.* diff --git a/README.md b/README.md index 7d42070..ac120de 100644 --- a/README.md +++ b/README.md @@ -2,7 +2,7 @@ [![CI](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/ci.yml/badge.svg)](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/ci.yml) [![Integration Tests](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/integration.yml/badge.svg)](https://github.com/MinaProtocol/mina-sdk-rust/actions/workflows/integration.yml) -[![codecov](https://codecov.io/gh/MinaProtocol/mina-sdk-rust/graph/badge.svg)](https://codecov.io/gh/MinaProtocol/mina-sdk-rust) +[![Coverage](https://coveralls.io/repos/github/MinaProtocol/mina-sdk-rust/badge.svg?branch=master)](https://coveralls.io/github/MinaProtocol/mina-sdk-rust?branch=master) [![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](LICENSE) Rust SDK for interacting with [Mina Protocol](https://minaprotocol.com) nodes via GraphQL.