LLFourn / bdk_core_staging

Staging area for bdk_core initial development
15 stars 12 forks source link

[cli] Remove Broadcast trait #198

Closed LLFourn closed 1 year ago

LLFourn commented 1 year ago

In favor of passing a closure that can do the broadcast. ~Broadcast was forcing us to create a "client" too early in the cli workflow~ this problem was orthogonal. It also forces us to wrap stuff in main.rs so we can implement the trait.

codecov-commenter commented 1 year ago

Codecov Report

Merging #198 (2a79a90) into master (0fb4e1b) will not change coverage. The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##           master     #198   +/-   ##
=======================================
  Coverage   56.74%   56.74%           
=======================================
  Files          10       10           
  Lines         252      252           
=======================================
  Hits          143      143           
  Misses        109      109           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.