Apply typo fixes from @tcharding

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

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.

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

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

[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

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`

Update changelog in preparation of `v0.7.0`

Update create transaction example code

The transaction builder changed a while ago, looks like some of the
example code did not get updated.

Update the transaction creation code to use a mutable builder.

Remove stale comments

The two fields this comment references are not `Option` type. This
comment seems to be stale.

Remove unneeded unit expression

Clippy emits:

  warning: unneeded unit expression

As suggested, remove the unneeded unit expression.

Clear clippy manual_map warning

The lint `manual_map` is new so we cannot explicitly allow it and
maintain backwards comparability. Instead, allow all lints for
`get_utxo_for` with a comment explaining why.

Use consistent field ordering

Clippy emits:

  warning: struct constructor field order is inconsistent with struct
  definition field order

As suggested, re-order the fields to be consistent with the struct
definition.

Use !any() instead of find()...is_none()

Clippy emits:

  warning: called `is_none()` after searching an `Iterator` with `find`

As suggested, use the construct: `!foo.iter().any(...)`

Remove unnecessary clone

Clippy emits:

  warning: using `clone` on type `descriptor::policy::Condition` which
  implements the `Copy` trait

Remove the clone and rely on `Copy`.

Use assert!(foo) instead of assert_eq!(foo, true)

It is redundant to pass true/false to `assert_eq!` since `assert!`
already asserts true/false.

This may, however, be controversial if someone thinks that

```
    assert_eq!(foo, false);
```

is more clear than

```
    assert!(!foo);
```

Use `assert!` directly instead of `assert_eq!` with true/false argument.

continue signing when finding already finalized inputs

Use PsbtKey instead of PSBT

We recently converted uses of `PSBT` -> `Psbt` inline with idiomatic
Rust acronym identifiers. Do the same to `PSBTKey`.

Use `PsbtKey` instead of `PSBTKey` when aliasing the import of
`psbt::raw::Key` from `bitcoin` library.

Use Psbt instead of PSBT

Idiomatic Rust uses lowercase for acronyms for all characters after the
first e.g. `std::net::TcpStream`. PSBT (Partially Signed Bitcoin
Transaction) should be rendered `Psbt` in Rust code if we want to write
idiomatic Rust.

Use `Psbt` instead of `PSBT` when aliasing the import of
`PartiallySignedTransaction` from `bitcoin` library.

continue signing when finding already finalized inputs

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

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

Use `default: D` mirroring Rust documentation

Currently we use `F: f` for the argument that is the default function
passed to `map_or_else` and pass a closure for the second argument. This
bent my brain while reading the documentation because the docs use
`default: D` for the first and `f: F` for the second. Although this is
totally trivial it makes deciphering the combinator chain easier if we
name the arguments the same way the Rust docs do.

Use `default: D` for the identifier of the default function passed into `map_or_else`.

get psbt inputs with bounds check

Check index out of bound also for tx inputs not only for psbt inputs

Bump version to 0.7.0-rc.1

Rewrite the `non_witness_utxo` check

[signer] Replace `force_non_witness_utxo` with `only_witness_utxo`

Instead of providing an opt-in option to force the addition of the
`non_witness_utxo`, we will now add them by default and provide the
option to disable them when they aren't considered necessary.

[signer] Adjust signing behavior with `SignOptions`

[signer] Fix error variant

Fix import for `FromStr`

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

Added grcov based code coverage reporting in github action

Make wallet methods take &mut psbt

Rather than consuming it because that is unergonomic.