Skip to content

Remove deprecated code and feature aliases#796

Closed
notmandatory wants to merge 5 commits intobitcoindevkit:masterfrom
notmandatory:remove_deprecated
Closed

Remove deprecated code and feature aliases#796
notmandatory wants to merge 5 commits intobitcoindevkit:masterfrom
notmandatory:remove_deprecated

Conversation

@notmandatory
Copy link
Copy Markdown
Member

@notmandatory notmandatory commented Nov 11, 2022

Description

A little housekeeping to remove recent and older deprecated code and feature aliases.

Notes to the reviewers

I also removed the descriptor checksum inception check and corresponding test since the ability to make bad checksums was deprecated and now removed.

Changelog notice

Removed deprecated

  • get_checksum_bytes and get_checksum functions
  • ConfirmationTime type
  • WalletExport type
  • Wallet.new_offline() function
  • Cargo feature aliases use-esplora-request and use-esplora-ureq

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@notmandatory notmandatory marked this pull request as draft November 11, 2022 17:42
@notmandatory notmandatory self-assigned this Nov 11, 2022
@notmandatory notmandatory marked this pull request as ready for review November 11, 2022 19:12
Copy link
Copy Markdown
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

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

Code review ACK.. Changes looks good to me.. One comment regarding the checksum inception bug..

Comment thread src/wallet/mod.rs
@notmandatory
Copy link
Copy Markdown
Member Author

This isn't urgent so moving to the 0.26 release

@notmandatory
Copy link
Copy Markdown
Member Author

Closing this, it's a lot to review and can be done as part of the bdk_core 1.0 release.

@notmandatory notmandatory deleted the remove_deprecated branch May 26, 2025 21:38
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.

2 participants