Fix clippy warn on compact_filters peer::_recv()

Fix clippy warning 'wrong_self_convention'

Fix clippy warning 'unnecessary_wraps'

Update version in the examples

Update CHANGELOG for release v0.4.0

Bump version to 0.4.0-rc.1

compact filters balance example

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

Properly handle the `Signet` network

Closes #62

[compact_filters] Use the new rust-bitcoin API

Update changelog

Update bitcoin, miniscript, electrum-client

Update bitcoin, miniscript, electrum-client

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

Un-pin the version of `cc`

Fixes #183

Emit specific compile error if incompatible features are enabled

This is motivated by the feature `electrum` being part of the
`default` features of this crate. It is easy to naively enable
`esplora` and `async-interface` and forget that `electrum` is enabled
by default, running into not so obvious compile errors.

Merge branch 'master' into make_txbuilder_take_ref_to_wallet

Fix whitespace and curse emacs

Fix CHANGELOG to mention s/utxos/add_utxos/

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

[keys] Improve the API of `DerivableKey`

A new `ExtendedKey` type has been added, which is basically an enum of
`bip32::ExtendedPubKey` and `bip32::ExtendedPrivKey`, with some extra metadata
regarding the `ScriptContext`.

This type has some methods that make it very easy to extract its content as
either an `xprv` or `xpub`.

The `DerivableKey` trait has been updated so that the user now only has to
implement a method (`DerivableKey::into_extended_key()`) to perform the
conversion into an `ExtendedKey`.

The method that was previously called `add_metadata()` has now been renamed
to `into_descriptor_key()`, and it has
a blanket implementation.

Make maintain_single_recipient return a Result

preferable to panicking.

[ci] Update rust toolchains

[signer] Add `Signer::id()`

Closes #261

Fix the crate version in `src/lib.rs`

Add a function to return the version of BDK at runtime

Merge branch 'master' into make_txbuilder_take_ref_to_wallet

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

Add "add_utxos" method on TxBuilder

To replace the previously existing ".utxos"

Remove not very useful comment

Thanks @tcharding.

Derive Clone + Debug for TxBuilder

And make Wallet Debug while I'm at it.

Revert back to Vec to hold utxos in builder

Due to brain malfunction I made utxos into a BTree. This made a test
pass but is incorrect. The test itself was incorrect as per comment in

https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-758370380

So I (1) reverted utxos back to a Vec, (2) fixed the test and expanded
the comment in the test.

Remove Option trickery from TxBuilder API

see: https://github.com/bitcoindevkit/bdk/pull/258#issuecomment-754685962

Assert that `.finish()` hasn't been called already in coin_selection

Add comment explaining why params and coin_selection are Options

Fix wrong doc links

Make testutils dependency path relative

Update Changelog for Tx creation overhaul

[wallet] Use doctest_wallet!() to remove some no_runs from doctests

...and improve the fee bumping example while trying to make it
no_run (but failed).

[wallet] Overhaul TxBuilder internals and externals

Fixes #251

TxBuilders are now not created directly but are created through the
wallet with `build_tx` and `build_fee_bump`.
The advantages of this realised in this commit are:

1. Normal tx creation and fee bumping use the code internally. The only
difference between normal tx and fee bump is how the builder is created.
2. The TxBuilder now has a refernce to the wallet and can therefore
lookup things as methods are called on it. `add_utxo` now uses this to
look up UTXO deta when it is called (rather than having to do it and
possibly error later on).

To support these changes `get_utxo` and `get_descriptor_for_keychain`
public methods have been added to Wallet. I could have kept them
pub(crate) but they seem like fine APIs to have publicly.

Change "received_tx" into "populate_test_db" macro

A `[cfg(test)]` function is not as helpful as a macro since it can't be
called in the context of a doctest.

Also adds doctest_wallet macro which can be used to create a wallet in a
doctest.

Merge branch 'release/0.3.0'

Bump version to 0.3.1-dev

Bump version to 0.3.0

Update CHANGELOG.md in preparation of tag `v0.3.0`

Use contains combinator

As suggested by clippy, use the `contains` combinator instead of doing
manual range check on floats.

Use contains combinator

As suggested by clippy, use the `contains` combinator instead of doing
manual range check on floats.

Add full stops to list items

Super anal patch to make list items uniform, add full stop to the items
where it is missing.

Remove unexplainable newlines

It seems the documentation of this project uses arbitrarily long
lines (i.e. no set column width) along with the occasional newline
before some sentences (within a paragraph). When to split a sentence
onto a newline does not seem to follow any discernible pattern.

There are a few instances of newline characters appearing randomly in
the middle of a sentence and since, as observed above, there is no
fixed column width is use these new lines are out of place.

Remove them so the documentation is slightly more uniform and nice to
read in an editor.

This patch is whitespace only, no other textual changes.

Do minor grammar fix

[wallet] Add "needed" and "available" metadata to Error::InsufficientFunds

Update CHANGELOG

[wallet] Add "needed" and "available" metadata to Error::InsufficientFunds

[blockchain] Upgrade tokio

- Also upgrade reqwest
- Switch to `tokio::runtime::Builder::new_single_thread()` because
`tokio::runtime::Runtime::new()` changed it's behavior to create a
multithreaded runtime.
- `enable_all` enables time and io resource drivers as explained
[here](https://docs.rs/tokio/0.2.24/tokio/runtime/index.html#resource-drivers)

Bump version to 0.3.0-rc.1

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

descriptor: Use `DescriptorError` instead of `Error` when reasonable

Change the return type of the `descriptor!()` macro and `ToWalletDescriptor` to
avoid having to map errors.

Also introduce more checks to validate descriptors built using the macro.

Permit to not set timeout in ElectrumBlockchainConfig

Allowing to use socks5 which requires None timeout

Fix crates.io license info

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

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

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

[ci] Fail 'Build docs' job if warnings

Remove 'cli.rs' module, 'cli-utils' feature and 'repl.rs' example

Fix clippy warnings for compact_filters feature

[ci] Fix clippy step to check matrix features

[wallet] Fix details.fees being wrong when change is dust

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

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

Use () to indicate a missing blockchain

So that:
1. There are no runtime errors
2. There less type annotations needed
3. Less traits and stuff to document

Fix incredibly annoying cargo-fmt problem

I must have a newer version of cargo-fmt which stops me from making
commits every time because of this.

[docs] Add badges for crates.io, mit license. Fix docs.rs badge and link

Fix the changelog after release v0.2.0

Bump version to 0.2.1-dev

Bump version to 0.2.0

Add metadata to Cargo.toml, remove local deps

Add metadata for `bdk-testutils` and `bdk-testutils-macros`, bump their version

[testutils-macros] Fix deps features

Add metadata for `bdk-macros`, bump its version

Document the development cycle

Bump version to 0.2.0-rc.1

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

[ci] Remove unneeded skip step conditionals in CI

Fix empty checkboxes in PR template

[docs] Format code in docs with '--config format_code_in_doc_comments=true'

[docs] Replace all 'allow(missing_docs)' with basic docs

[docs] Add more docs to 'types.rs'

[keys] impl `ToDescriptorKey` for `&str`

[descriptor] Make the syntax of `descriptor!()` more consistent

The syntax now is pretty much the same as the normal descriptor syntax,
with the only difference that modifiers cannot be grouped together (i.e.
`sdv:older(144)` must be turned into `s:d:v:older(144)`.

[docs] use only sled instead of crate::sled

[docs] fix NetworkMessage::Ping docs link

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

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

always build docs and create artifacts, publish only on master

[docs] Add docs to the 'wallet' module

[ci] Fix publishing coverage to codecov.io

Take ID into account in SignersContainerKey's PartialEq impl

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

[wallet] Add tests for `check_nsequence_rbf` and `check_nlocktime`

[wallet] Set the correct nSequence when RBF and OP_CSV are used

This commit also fixes the timelock comparing logic in the policy module, since
the rules are different for absolute (OP_CLTV) and relative (OP_CSV) timelocks.

Fixes #215