Skip to content

CI: Replace Coveralls with Codecov for coverage reporting#247

Merged
tvpeter merged 1 commit intobitcoindevkit:masterfrom
tvpeter:ci/codecov
Mar 10, 2026
Merged

CI: Replace Coveralls with Codecov for coverage reporting#247
tvpeter merged 1 commit intobitcoindevkit:masterfrom
tvpeter:ci/codecov

Conversation

@tvpeter
Copy link
Collaborator

@tvpeter tvpeter commented Mar 9, 2026

Description

This PR updates the code_coverage workflow by replacing the Coveralls code coverage reports with codecov. It also updates the checkout, rust-lang toolchain setup actions.
Fixes #245

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

- update checkout to v6
- update rust toolchain to use official action
- replace coveralls with codecov
@codecov
Copy link

codecov bot commented Mar 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 10.86%. Comparing base (6f61d2e) to head (a675935).
⚠️ Report is 115 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (6f61d2e) and HEAD (a675935). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (6f61d2e) HEAD (a675935)
1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #247       +/-   ##
===========================================
- Coverage   55.87%   10.86%   -45.02%     
===========================================
  Files           4        8        +4     
  Lines         995     2466     +1471     
===========================================
- Hits          556      268      -288     
- Misses        439     2198     +1759     
Flag Coverage Δ
rust 10.86% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK a675935

@tvpeter tvpeter merged commit 15b6f5b into bitcoindevkit:master Mar 10, 2026
8 of 9 checks passed
@github-project-automation github-project-automation bot moved this to Done in BDK-CLI Mar 10, 2026
@tvpeter tvpeter added this to the CLI 3.0.0 milestone Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

ci: replace coveralls with codecov server

2 participants