Skip to content

Add Error::SerdeJson variant and remove unwraps on submit_package#171

Merged
oleonardolima merged 4 commits intobitcoindevkit:masterfrom
luisschwab:feat/serde-json-err-variant
Feb 12, 2026
Merged

Add Error::SerdeJson variant and remove unwraps on submit_package#171
oleonardolima merged 4 commits intobitcoindevkit:masterfrom
luisschwab:feat/serde-json-err-variant

Conversation

@luisschwab
Copy link
Member

@luisschwab luisschwab commented Feb 11, 2026

Closes #161.
Replaces #163.

This PR adds a new error variant for serde_json and removes the lingering unwraps on the submit_package methods.

Changelog Notice

# Added
- New `Error::SerdeJson` error variant

Also bumps electrsd to v0.36.1 and fixes MSRV dependency pinning.

@luisschwab luisschwab self-assigned this Feb 11, 2026
@luisschwab luisschwab changed the title Add Error::SerdeJson variant and remove unwraps on submit_package' Add Error::SerdeJson variant and remove unwraps on submit_package Feb 11, 2026
@luisschwab
Copy link
Member Author

@oleonardolima we'll need to bump electrsd

@coveralls
Copy link

coveralls commented Feb 11, 2026

Pull Request Test Coverage Report for Build 21926755551

Details

  • 0 of 6 (0.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 86.757%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/blocking.rs 0 1 0.0%
src/async.rs 0 5 0.0%
Totals Coverage Status
Change from base Build 21211295857: 0.05%
Covered Lines: 1402
Relevant Lines: 1616

💛 - Coveralls

@luisschwab luisschwab force-pushed the feat/serde-json-err-variant branch from f8e8492 to ee70e95 Compare February 11, 2026 21:47
@luisschwab luisschwab force-pushed the feat/serde-json-err-variant branch from ee70e95 to e93c1b1 Compare February 11, 2026 22:15
Copy link
Collaborator

@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.

utACK e93c1b1

@luisschwab luisschwab force-pushed the feat/serde-json-err-variant branch from e93c1b1 to d651e71 Compare February 11, 2026 23:11
Copy link
Collaborator

@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 d651e71

@oleonardolima oleonardolima merged commit 8187765 into bitcoindevkit:master Feb 12, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add serde_json Error variant

3 participants