bitcoindevkit / bdk / 12602977761

79%

push

github

Merge bitcoindevkit/bdk#1592: Architectural Decision Records

<a class=hub.com/bitcoindevkit/bdk/commit/3b10abbef69a2245714a762a555ff58a67217eec">3b10abbef<a href="https://github.com/bitcoindevkit/bdk/commit/abc305612160c0e2ce85c7bba4c2c162ff488adc"> docs: add ADR 0002_persisted.md (valued mammal)
<a class="double-link" href="https://github.com/bitcoindevkit/bdk/commit/1c4b24469a971fdb2490b7358b10f9c480a7c16a">1c4b24469</a><a href="https://github.com/bitcoindevkit/bdk/commit/abc305612160c0e2ce85c7bba4c2c162ff488adc"> docs: add ADR 0001_persist.md (valued mammal)
</a><a class="double-link" href="https://github.com/bitcoindevkit/bdk/commit/e60c65b484fb8a3754b206b97cc3ef4a4d2f6334">e60c65b48</a> docs: add architectural decision record (ADR) template (valued mammal)

Pull request description:

  Opening this up as a record of past decisions. The template is inspired by the one used by uniffi. Whether or not we decide to add these to the repo, it's an opportunity to learn, discuss, and collab when it comes to design and architecture.

  The first 2 ADRs pertain to bdk's approach to data persistence. Another recent suggestion was to document how we're thinking about single vs multiple descriptor wallets. Feedback welcome.

  fixes #1309

ACKs for top commit:
  thunderbiscuit:
    ACK 3b10abbef69a2245714a762a555ff58a67217eec.

Tree-SHA512: 9eeed764a30295df8b56355f5ff1a19b7d4

11971 of 14275 relevant lines covered (83.86%)

20244.26 hits per line