Description

Closes #94.

This PR replaces bitcoin::Network for bitcoin::NetworkKind where it applies.

I also took the liberty of fixing up and adding comments on the files I touched.

Changelog

• wallet_name_from_descriptor() takes NetworkKind instead of Network.
• FnOnce takes NetworkKind instead of Network.
• impl IntoWalletDescriptor for <T> takes Network instead NetworkKind.
• DescriptorTemplate::build() takes NetworkKind instead of Network.
• ExtendedKey::into_xprv() takes NetworkKind instead of Network.
• ExtendedKey::into_xpub() takes NetworkKind instead of Network.
• DerivableKey::into_extended_key() examples updated to use NetworkKind instead of Network.
• any_network() renamed to any_network_kind().
• mainnet_network() renamed to mainnet_network_kind().
• test_networks() renamed to test_network_kind().
• merge_networks() renamed to merge_network_kinds().
• ValidNetworks type alias renamed to ValidNetworkKinds.
• GeneratedKey::new() takes ValidNetworkKinds instead of ValidNetworks.
• DescriptorKey::from_public() takes ValidNetworkKinds instead of ValidNetworks.
• DescriptorKey::from_secret() takes ValidNetworkKinds instead of ValidNetworks.
• KeyError::InvalidNetwork renamed to KeyError::InvalidNetworkKind.
• DescriptorKey::override_valid_networks() renamed to override_valid_network_kinds().
• Network validation logic updated to use NetworkKind instead of Network.
• Test network validation now uses NetworkKind::Test instead of Network::{Regtest, Signet, Testnet, Testnet4}.
• Examples and comments updated to use NetworkKind instead of Network.

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo +nightly fmt and cargo clippy before committing

New Features:

• I've added tests for the new feature
• I've added docs for the new feature

Bugfixes:

• This pull request breaks the existing API
• I've added tests to reproduce the issue which are now passing
• I'm linking the issue being fixed by this PR