Skip to content

Commit

Permalink
Merge #1722: fix(testenv): disable downloads (bitcoind and electrsd) …
Browse files Browse the repository at this point in the history
…for docs.rs b…

9b48fd4 fix(testenv): disable downloads (bitcoind and electrsd) for docs.rs builds of crate testenv (River Kanies)

Pull request description:

  …uilds of crate testenv

  ### Description

  address this issue #1627
  where docs.rs build is failing for bdk_testenv crate

  original PR: #1679

  ### Notes to the reviewers

  I was not able to reproduce the build issue locally, so this will have to be tested live unless someone else can reproduce the build error https://docs.rs/crate/bdk_testenv/0.10.0/builds/1377651

  more details in this thread on discord https://discord.com/channels/753336465005608961/1265333904324427849/1304476756660719668

  #### Bugfixes:

  * [x] I'm linking the issue being fixed by this PR

ACKs for top commit:
  ValuedMammal:
    ACK 9b48fd4 but I guess we won't know for sure until we publish the crate
  notmandatory:
    ACK 9b48fd4

Tree-SHA512: fe4ad420f71530a279e1cf86af6e7bd59f381804a5c7c9636fa2aaa5592ff4e6fed668513831d7d63ed0a97152d685b187d50e906b21dbbb2180dedf5d28fad0
  • Loading branch information
notmandatory committed Nov 21, 2024
2 parents 3bc45b5 + 9b48fd4 commit c42b01b
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 5 deletions.
2 changes: 1 addition & 1 deletion crates/bitcoind_rpc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ bitcoincore-rpc = { version = "0.19.0" }
bdk_core = { path = "../core", version = "0.3.0", default-features = false }

[dev-dependencies]
bdk_testenv = { path = "../testenv", default-features = false }
bdk_testenv = { path = "../testenv" }
bdk_chain = { path = "../chain" }

[features]
Expand Down
2 changes: 1 addition & 1 deletion crates/electrum/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ bdk_core = { path = "../core", version = "0.3.0" }
electrum-client = { version = "0.21", features = [ "proxy" ], default-features = false }

[dev-dependencies]
bdk_testenv = { path = "../testenv", default-features = false }
bdk_testenv = { path = "../testenv" }
bdk_chain = { path = "../chain" }

[features]
Expand Down
2 changes: 1 addition & 1 deletion crates/esplora/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ miniscript = { version = "12.0.0", optional = true, default-features = false }

[dev-dependencies]
bdk_chain = { path = "../chain" }
bdk_testenv = { path = "../testenv", default-features = false }
bdk_testenv = { path = "../testenv" }
tokio = { version = "1", features = ["rt", "rt-multi-thread", "macros"] }

[features]
Expand Down
11 changes: 9 additions & 2 deletions crates/testenv/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,16 @@ workspace = true

[dependencies]
bdk_chain = { path = "../chain", version = "0.20.0", default-features = false }
electrsd = { version = "0.28.0", features = [ "bitcoind_25_0", "esplora_a33e97e1", "legacy" ] }
electrsd = { version = "0.28.0", features = [ "legacy" ], default-features = false }

[dev-dependencies]
bdk_testenv = { path = "." }

[features]
default = ["std"]
default = ["std", "download"]
download = ["electrsd/bitcoind_25_0", "electrsd/esplora_a33e97e1"]
std = ["bdk_chain/std"]
serde = ["bdk_chain/serde"]

[package.metadata.docs.rs]
no-default-features = true

0 comments on commit c42b01b

Please sign in to comment.