Skip to content

ci: individual package tests#2209

Open
ValuedMammal wants to merge 1 commit into
bitcoindevkit:masterfrom
ValuedMammal:update_ci
Open

ci: individual package tests#2209
ValuedMammal wants to merge 1 commit into
bitcoindevkit:masterfrom
ValuedMammal:update_ci

Conversation

@ValuedMammal

@ValuedMammal ValuedMammal commented May 19, 2026

Copy link
Copy Markdown
Collaborator

Close #1944

Description

Replaces the monolithic build-test matrix job (which ran a 2-toolchain × 2-feature-set matrix across the whole workspace) with dedicated per-crate build jobs. Each crate uses named steps for every meaningful feature combination, making it easy to identify which specific crate and feature set broke in CI.

Changes:

  • Per-crate jobs: build-core, build-chain, build-bitcoind-rpc, build-electrum, build-esplora, build-file-store, build-testenv — each with per-feature cargo build steps and a cargo test --all-features step
  • msrv job: Simplified from a matrix to a single flat job targeting 1.85.0; runs cargo build --workspace --all-targets and cargo test --workspace --lib --tests --all-features
  • check-no-std: Updated to check bdk_core and bdk_chain with no default features + with hashbrown and miniscript; cross-compilation against thumbv6m-none-eabi is stubbed out as usual to be addressed in a separate PR

@codecov

codecov Bot commented May 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.69%. Comparing base (47556ab) to head (a96fc85).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2209   +/-   ##
=======================================
  Coverage   77.69%   77.69%           
=======================================
  Files          29       29           
  Lines        5801     5801           
  Branches      271      271           
=======================================
  Hits         4507     4507           
  Misses       1223     1223           
  Partials       71       71           
Flag Coverage Δ
rust 77.69% <ø> (ø)

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

☔ View full report in Codecov by Harness.
📢 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.

@ValuedMammal ValuedMammal self-assigned this May 19, 2026
@ValuedMammal ValuedMammal added the github_actions Pull requests that update GitHub Actions code label May 19, 2026
@ValuedMammal ValuedMammal moved this to In Progress in BDK Chain May 19, 2026
@ValuedMammal ValuedMammal force-pushed the update_ci branch 2 times, most recently from adeded1 to 06a3fe4 Compare May 26, 2026 13:18
@ValuedMammal

Copy link
Copy Markdown
Collaborator Author
  • electrum/benches/test_sync.rs requires use-rustls feature
  • ci: Changed Build 1.85.0 (no default features) to cargo build --workspace --all-targets --no-default-features

Added workspace level MSRV job.

Added per-crate jobs to build meaningful feature
combinations and test all features.

Removed build-test job.
@ValuedMammal ValuedMammal marked this pull request as ready for review June 10, 2026 16:44
@oleonardolima oleonardolima added this to the Chain 0.24.0 milestone Jun 11, 2026
@oleonardolima oleonardolima moved this from In Progress to Needs Review in BDK Chain Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

Status: Needs Review

Development

Successfully merging this pull request may close these issues.

Add CI tests for building individual packages

2 participants