Ran
|
Jobs
1
|
Files
44
|
Run time
1min
|
Badge
Embed ▾
README BADGES
|
push
github
Merge bitcoindevkit/bdk#1473: Remove `persist` submodule a0bf45bef docs: remove PersistBackend from docs, Cargo.toml and wallet README.md (Steve Myers) feb27df18 feat(chain)!: add `take` convenience method to `Append` trait (志宇) 1eca568be feat!: rm `persist` submodule (志宇) Pull request description: ### Description @LLFourn suggested these changes which greatly simplifies the amount of code we have to maintain and removes the `async-trait` dependency. We remove `PersistBackend`, `PersistBackendAsync`, `StageExt` and `StageExtAsync` completely. Instead, we introduce `Wallet::take_staged(&mut self) -> Option<ChangeSet>`. The original intention to keep a staging area (`StageExt`, `StageExtAsync`) is to enforce: 1. The caller will not persist an empty changeset. 2. The caller does not take the staged changeset if the database (`PersistBackend`) fails. We achieve `1.` by returning `None` if the staged changeset is empty. `2.` is not too important. The caller can try figure out what to do with the changeset if persisting to db fails. ### Notes to the reviewers I added a `take` convenience method to the `Append` trait. I thought it would be handy for the caller to do a staging area with this. ### Changelog notice * Remove `persist` submodule from `bdk_chain`. * Change `Wallet` to outsource it's persistence logic by introducing `Wallet::take_staged`. * Add `take` convenience method to `Append` trait. ### Checklists #### All Submissions: * [x] I've signed all my commits * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md) * [x] I ran `cargo fmt` and `cargo clippy` before committing #### New Features: ~* [ ] I've added tests for the new feature~ * [x] I've added docs for the new feature ACKs for top commit: notmandatory: ACK a0bf45bef1b53f8aec273f7... (continued)
58 of 64 new or added lines in 5 files covered. (90.63%)
1 existing line in 1 file now uncovered.11128 of 13357 relevant lines covered (83.31%)
17144.51 hits per line
Lines | Coverage | ∆ | File |
---|---|---|---|
1 |
97.73 |
crates/chain/src/changeset.rs | |
1 |
61.54 |
2.71% | crates/chain/src/tx_data_traits.rs |
4 |
82.93 |
0.48% | crates/wallet/src/wallet/mod.rs |
Lines | Coverage | ∆ | File |
---|---|---|---|
1 |
82.93 |
0.48% | crates/wallet/src/wallet/mod.rs |
ID | Job ID | Ran | Files | Coverage | |
---|---|---|---|---|---|
1 | 9524710957.1 | 44 |
83.31 |
GitHub Action Run |
Coverage | ∆ | File | Lines | Relevant | Covered | Missed | Hits/Line |
---|