Skip to content

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

Draft
ValuedMammal wants to merge 3 commits into
bitcoindevkit:masterfrom
ValuedMammal:edition_2024
Draft

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

Conversation

@ValuedMammal
Copy link
Copy Markdown
Collaborator

Description

Notes to the reviewers

WIP: Updates the rust edition in Cargo.toml to 2024.

Changelog notice

Checklists

All Submissions:

  • I ran just p before pushing

@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
Copy link
Copy Markdown

codecov Bot commented Mar 31, 2026

Codecov Report

❌ Patch coverage is 35.29412% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.52%. Comparing base (6dca318) to head (56f4acb).

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   80.55%   80.52%   -0.03%     
==========================================
  Files          24       24              
  Lines        5472     5474       +2     
  Branches      243      243              
==========================================
  Hits         4408     4408              
- Misses        987      989       +2     
  Partials       77       77              
Flag Coverage Δ
rust 80.52% <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 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.

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.
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: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant