rust-bitcoin / rust-bitcoin / 16143399170

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4684: bitcoin: Remove re-export of Denomination

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/496d7df77ac45e23d11218e05084711356e01543">496d7df77 bitcoin: Remove re-export of Denomination (Tobin C. Harding)

Pull request description:

  `units::Denomination` does not exist, it is only at `units::amount::Denomination`. Our policy is to make the following three equivalent `bitcoin::Foo`, `primitives::Foo`, and `units::Foo` therefor we should not re-export `Denomination` at the crate root of `bitcoin`.


ACKs for top commit:
  apoelstra:
    ACK 496d7df77ac45e23d11218e05084711356e01543; successfully ran local tests; will one-ACK merge; seems trivially correct


Tree-SHA512: b16f0b4bddddc55bf40b97434abe7c7359c

21301 of 25488 relevant lines covered (83.57%)

15280.22 hits per line