Skip to content

Replace std::error::Error with core::error::Error#397

Merged
ValuedMammal merged 1 commit intobitcoindevkit:masterfrom
luisschwab:chore/use-core-error
Mar 9, 2026
Merged

Replace std::error::Error with core::error::Error#397
ValuedMammal merged 1 commit intobitcoindevkit:masterfrom
luisschwab:chore/use-core-error

Conversation

@luisschwab
Copy link
Member

@luisschwab luisschwab commented Mar 6, 2026

The Error trait is stable as of 1.81.0, so import it from core instead.

Changelog notice

# Changed

- Implement `core::error::Error` for several types and un-feature-gate `std::error::Error`.

Checklists

All Submissions:

@luisschwab luisschwab self-assigned this Mar 6, 2026
@luisschwab luisschwab added the chore Non-coding related work label Mar 6, 2026
@luisschwab luisschwab moved this to Needs Review in BDK Wallet Mar 6, 2026
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.77%. Comparing base (c4a3b8c) to head (fb9b0b6).
⚠️ Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #397      +/-   ##
==========================================
+ Coverage   79.06%   79.77%   +0.70%     
==========================================
  Files          24       24              
  Lines        5298     5266      -32     
  Branches      241      241              
==========================================
+ Hits         4189     4201      +12     
+ Misses       1032      988      -44     
  Partials       77       77              
Flag Coverage Δ
rust 79.77% <ø> (+0.70%) ⬆️

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.

@luisschwab luisschwab changed the title Replace std::error::Error with core::error::Error Replace std::error::Error with core::error::Error Mar 6, 2026
Copy link
Contributor

@110CodingP 110CodingP left a comment

Choose a reason for hiding this comment

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

ACK 381b425

Copy link
Contributor

@nymius nymius left a comment

Choose a reason for hiding this comment

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

cACK 381b425

The `Error` trait is stable as of 1.81.0, so import it from `core` instead.
@luisschwab luisschwab force-pushed the chore/use-core-error branch from 381b425 to fb9b0b6 Compare March 6, 2026 14:56
@luisschwab luisschwab requested review from 110CodingP and nymius March 6, 2026 14:57
Copy link
Contributor

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

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

ACK fb9b0b6

I agree that the FromStr can be addressed in a follow-up.

Please add the CHANGELOG notice in the PR description, also I think this can land in the 3.0 release, wdyt @ValuedMammal ?.

@oleonardolima oleonardolima added this to the Wallet 3.0.0 milestone Mar 6, 2026
Copy link
Contributor

@nymius nymius left a comment

Choose a reason for hiding this comment

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

ACK fb9b0b6

I'm not sure about adding this to the CHANGELOG. Is this noticeable enough?

@luisschwab
Copy link
Member Author

For the user this is a no-op, not sure if this belongs in there. But I could be convinced otherwise.

Copy link
Contributor

@110CodingP 110CodingP left a comment

Choose a reason for hiding this comment

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

reACK fb9b0b6

FWIW I also agree that this and #390 would be nice haves for 3.0 .

@ValuedMammal
Copy link
Collaborator

ACK fb9b0b6

@ValuedMammal ValuedMammal merged commit c972ef2 into bitcoindevkit:master Mar 9, 2026
17 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Wallet Mar 9, 2026
@luisschwab luisschwab deleted the chore/use-core-error branch March 9, 2026 14:56
This was referenced Mar 9, 2026
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

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants