-
Notifications
You must be signed in to change notification settings - Fork 412
Closed
Labels
good first issueGood for newcomersGood for newcomers
Description
As suggested in the code base:
Lines 14 to 17 in adf7d0c
| // TODO: maybe write our own implementation of bip39? Seems stupid to have an extra dependency for | |
| // something that should be fairly simple to re-implement. | |
| use bitcoin::util::bip32; |
I also recommend writing own BIP39 implementation.
In issue #399, BDK transitioned from tiny-bip39 to rust-bip39, however, the latter package (which BDK now depends on) appears to be unmaintained or poorly maintained, with PRs and issues ignored for about a year.
https://github.com/rust-bitcoin/rust-bip39
At a minimum, BIP39 wordlists, and words -> entropy -> seed generation would be valuable.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomers