Skip to content

chore: Update rust edition to 2024 in Cargo.toml - #425

Merged
ValuedMammal merged 3 commits into
bitcoindevkit:masterfrom
ValuedMammal:edition_2024
Jul 27, 2026
Merged

chore: Update rust edition to 2024 in Cargo.toml#425
ValuedMammal merged 3 commits into
bitcoindevkit:masterfrom
ValuedMammal:edition_2024

Conversation

@ValuedMammal

@ValuedMammal ValuedMammal commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Description

This PR updates bdk_wallet to Rust edition 2024 and includes the related formatting changes needed to keep the codebase consistent with the new edition style. The changes are intended to be mechanical, with no functional behavior changes or API impact.

Changelog notice

Changed

  • chore: Update Rust edition to 2024

Checklists

Before submitting

@ValuedMammal ValuedMammal self-assigned this Mar 31, 2026
@ValuedMammal ValuedMammal added the chore Non-coding related work label Mar 31, 2026
@ValuedMammal ValuedMammal added this to the Wallet 4.0.0 milestone Mar 31, 2026
@ValuedMammal ValuedMammal moved this to In Progress in BDK Wallet Mar 31, 2026
@codecov

codecov Bot commented Mar 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 35.29412% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.77%. Comparing base (e6e3ddf) to head (72a95f8).

Files with missing lines Patch % Lines
src/wallet/signer.rs 0.00% 4 Missing ⚠️
src/descriptor/policy.rs 40.00% 3 Missing ⚠️
src/wallet/error.rs 0.00% 2 Missing ⚠️
src/descriptor/mod.rs 50.00% 1 Missing ⚠️
src/wallet/mod.rs 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #425      +/-   ##
==========================================
- Coverage   81.79%   81.77%   -0.03%     
==========================================
  Files          25       25              
  Lines        6482     6484       +2     
  Branches      296      296              
==========================================
  Hits         5302     5302              
- Misses       1081     1083       +2     
  Partials       99       99              
Flag Coverage Δ
rust 81.77% <35.29%> (-0.03%) ⬇️

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.

This is to reduce noise related to formatting when we change
the rust edition to 2024.
Removed explicit `ref` binding modifiers, as otherwise
rustc complains with "cannot explicitly borrow within an
implicitly borrowing pattern." Refer to
https://doc.rust-lang.org/reference/patterns.html#binding-modes
for details.
Removed style_edition from rustfmt.toml
and ran cargo +nightly fmt.
@ValuedMammal
ValuedMammal marked this pull request as ready for review July 23, 2026 17:22
@ValuedMammal ValuedMammal moved this from In Progress to Needs Review in BDK Wallet Jul 23, 2026

@oleonardolima oleonardolima left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ACK 72a95f8

@ValuedMammal

Copy link
Copy Markdown
Contributor Author

Thanks @oleonardolima

@ValuedMammal
ValuedMammal merged commit 20349c7 into bitcoindevkit:master Jul 27, 2026
12 of 14 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in BDK Wallet Jul 27, 2026
@ValuedMammal
ValuedMammal deleted the edition_2024 branch July 27, 2026 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Non-coding related work

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants