bitcoindevkit / bdk / 22785574568

78%

push

github

Merge bitcoindevkit/bdk#2120: fix(chain): forward `confirmation_height_upper_bound` in `Anchor` implementation for `&A`

<a class=hub.com/bitcoindevkit/bdk/commit/<a class="double-link" href="https://git"><a class=hub.com/bitcoindevkit/bdk/commit/5d97e136ae89854c53add90d9da9131fec56c4a0">5d97e136a fix(chain): forward `confirmation_height_upper_bound` in `Anchor` impl for `&A` (志宇)

Pull request description:

  ### Description

  The blanket `Anchor` impl for `&A` was missing the `confirmation_height_upper_bound` method, causing it to fall back to the default implementation instead of delegating to the inner type.

  ### Changelog notice

  ```md
  Fixed:

  - The `Anchor::confirmation_height_upper_bound` impl was missing for `&A`, causing it to fallback to the default impl.
  ```

  ### Checklists

  #### All Submissions:

  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)

  #### Bugfixes:

  ~~* [ ] This pull request breaks the existing API~~
  ~~* [ ] I've added tests to reproduce the issue which are now passing~~
  ~~* [ ] I'm linking the issue being fixed by this PR~~

ACKs for top commit:
  ValuedMammal:
    ACK 5d97e136ae89854c53add90d9da9131fec56c4a0
  Dmenec:
    tACK 5d97e136ae89854c53add90d9da9131fec56c4a0

Tree-SHA512: baea97e5a7914c56a42427c1248d731e56b

376 of 510 branches covered (73.73%)

Branch coverage included in aggregate %.

0 of 3 new or added lines in 1 file covered. (0.0%)

4293 of 5465 relevant lines covered (78.55%)

3175.72 hits per line