Merge commit 'refs/pull/371/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/393/head' of github.com:bitcoindevkit/bdk

Apply typo fixes from @tcharding

Co-authored-by: Tobin C. Harding <me@tobin.cc>

Merge commit 'refs/pull/388/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/375/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/389/head' of github.com:bitcoindevkit/bdk

[docs] clarify when the fee could be unknown

Replace set_single_recipient with drain_to

What set_single_recipient does turns out to be useful with multiple
recipients.
Effectively, set_single_recipient was simply creating a change
output that was arbitrarily required to be the only output.
But what if you want to send excess funds to one address but still have
additional recipients who receive a fixed value?
Generalizing this to `drain_to` simplifies the logic and removes several
error cases while also allowing new use cases.

"maintain_single_recipient" is also replaced with "allow_shrinking"
which has more general semantics.

Replace set_single_recipient with drain_to

What set_single_recipient does turns out to be useful with multiple
recipients.
Effectively, set_single_recipient was simply creating a change
output that was arbitrarily required to be the only output.
But what if you want to send excess funds to one address but still have
additional recipients who receive a fixed value?
Generalizing this to `drain_to` simplifies the logic and removes several
error cases while also allowing new use cases.

"maintain_single_recipient" is also replaced with "allow_shrinking"
which has more general semantics.

[rpc] Show in the docs that the RPC APIs are feature-gated

Bump version to 0.9.1-dev

Bump version to 0.9.0

Update version in `src/lib.rs`

Update CHANGELOG

Add Vbytes trait

We convert weight units into vbytes in various places. Lets add a trait
to do it, this makes the code slightly cleaner.

Merge commit 'refs/pull/383/head' of github.com:bitcoindevkit/bdk into release/0.9.0

Merge commit 'refs/pull/383/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/385/head' of github.com:bitcoindevkit/bdk into release/0.9.0

rustfmt

dsl: add regression test for and_or() descriptor

dsl: fix descriptor macro when and_or() used

Build the `rpc` feature on docs.rs

[rpc] Mark the RPC backend as experimental

Build the `rpc` feature on docs.rs

[rpc] Mark the RPC backend as experimental

Bump version to 0.9.0-rc.1

[verify] Build the `verify` feature on docs.rs

[verify] Use `impl_error!()` whenever possible

[verify] Add documentation

[verify] Cache txs to avoid multiple db/network lookups

[wallet] Verify unconfirmed transactions after syncing

Verify the unconfirmed transactions we download against the consensus
rules. This is currently exposed as an extra `verify` feature, since it
depends on a pre-release version of `bitcoinconsensus`.

Closes #352

[wallet] Verify unconfirmed transactions after syncing

Verify the unconfirmed transactions we download against the consensus
rules. This is currently exposed as an extra `verify` feature, since it
depends on a pre-release version of `bitcoinconsensus`.

Closes #352

fix readme

update github ci removing electrs download and fixing cache

use electrsd with feature to download the binary

use bitcoind with feature to download the binary

use cfg! instead of #[cfg] and use semver

Remove container and test blockchains downloading backends executables

trigger electrs when polling

remove serial_test

use bitcoind and electrsd crate to launch daemons

bump bitcoind dep to 0.11.0

[testutils] Allow the generated blockchain tests to access `test_client`

add Copy trait to Progress types

Bump CI stable version to `1.53`

Fixes #374

Fix clippy warnings on `1.53`

Fix `clippy::inconsistent_struct_constructor`: the constructor field
order was inconsistent with the struct declaration.

Fix CHANGELOG

The `Rpc` backend is not part of the release but it accidentally ended
up there during the merge

Merge commit 'refs/pull/370/head' of github.com:bitcoindevkit/bdk

[ci] Revert change to run_blockchain_tests.sh back to using container id

[ci] Change blockchain tests to use bitcoind rpc cookie authentication

Merge branch 'release/0.8.0'

Fix CHANGELOG unreleased link

Add CHANGELOG v0.8.0 link

make fee in TransactionDetails Option, add confirmation_time field as Option

confirmation_time contains both a block height and block timestamp and is
Some only for confirmed transaction

Merge commit 'refs/pull/348/head' of github.com:bitcoindevkit/bdk

Bump version to 0.8.1-dev

Bump version to 0.8.0

Update CHANGELOG

Bump version in `src/lib.rs`

Use bitcoin's base64 feature for Psbts

Remove redundant borrows

Clippy emits:

  warning: this expression borrows a reference

As suggested remove the borrows from the front of vars that are already references.

Merge commit 'refs/pull/365/head' of github.com:bitcoindevkit/bdk into release/0.8.0

Use no_run instead of ignore

We have an attribute `no_run` that builds but does not run example code
in Rustdocs, this keeps the examples building as the codebase evolves.

use `no_run` and fix example code so it builds cleanly during test run.

Some examples that require the `electrum` feature to be available have
been feature-gated to make sure they aren't accidentally compiled when
that feature is not enabled.

Co-authored-by: Alekos Filini <alekos.filini@gmail.com>

Remove explicit `feature(external_doc)`

It looks like this is now enabled by default as of `cargo 1.54.0-nightly (0cecbd673 2021-06-01)`

Fix example/doctests format

[ci] Check fmt for examples/doctests

add details to TODO, format doc example

Bump version to 0.8.0-rc.1

move scan in setup

update bitcoind dep

use storage address instead of satoshi's

always ask node for tx no matter capabilities

correctly initialize UTXO keychain kind

change feature to execute sync from rpc to test-rpc

rename struct to CallResult

match also on signet

update changelog with rpc feature

Implements RPC Backend

Use AddressInfo in private methods

return AddressInfo from get_address

[signer] Add an option to explicitly allow using non-`ALL` sighashes

Instead of blindly using the `sighash_type` set in a psbt input, we
now only sign `SIGHASH_ALL` inputs by default, and require the user to
explicitly opt-in to using other sighashes if they desire to do so.

Fixes #350

Merge commit 'refs/pull/347/head' of github.com:bitcoindevkit/bdk

Initialize env_logger at start of blockchain tests

run_blockchain_tests.sh improvements

Uppercase 'Test' so that github can see what's up

It is expecting something named 'Test electrum'

Add script for running the blockchain tests locally

Fix clippy warnings inside testutils macro

Now that it's inside the main repo clippy is having a go at me.

Remove testutils feature so doctests worka again

I wanted to only conditionally compile testutils but it's needed in
doctests which we can't conditionally compile for:

https://github.com/rust-lang/rust/issues/67295

Move blockchain related stuff to blockchain_tests

Merge testutils crate into the main crate

This avoids having to keep the apis in sync between the macros and the
main project.

Run blockchain tests on esplora

They were only being run on electrum before.

Merge commit 'refs/pull/338/head' of github.com:bitcoindevkit/bdk

Roll blockchain tests proc macro into normal macro

This means one less crate in the repo. Had to do a Default on TestClient
to satisfy clippy.

Merge commit 'refs/pull/346/head' of github.com:bitcoindevkit/bdk

Fix changelog

Bump version to 0.7.1-dev

Bump version to 0.7.0

Use the released `testutils-macros`

Bump version in `src/lib.rs`

Bump `testutils-macros` to `v0.6.0`