bitcoindevkit / bdk / 12108281623

79%

push

github

Merge bitcoindevkit/bdk#1733: feat(chain,wallet)!: Transitive `ChainPosition`

<a class=hub.com/bitcoindevkit/bdk/commit/29b374e383bc9317b55a2a58019e1948e170fabd">29b374e38 feat(chain,wallet)!: Transitive `ChainPosition` (志宇)

Pull request description:

  ### Description

  Change `ChainPosition` to be able to represent transitive anchors and unconfirm-without-last-seen values.

  As mentioned in https://github.com/bitcoindevkit/bdk/pull/1670#issuecomment-2484535021, we want this merged first so that we have minimal changes to the API after 1670 is merged.

  ### Changelog notice

  * Change `ChainPosition` so that it is able to represent transitive anchors and unconfirmed-without-last-seen values.

  ### 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:
  ValuedMammal:
    ACK 29b374e383bc9317b55a2a58019e1948e170fabd

Tree-SHA512: 58f22f382cde6dfcd10fb5165eddaa1acc6

124 of 126 new or added lines in 7 files covered. (98.41%)

11671 of 13990 relevant lines covered (83.42%)

17788.19 hits per line