]> Untitled Git - bdk/commitdiff
Merge bitcoindevkit/bdk#1425: chore: add `print_stdout`/`print_stderr` lints to works...
authorSteve Myers <steve@notmandatory.org>
Thu, 29 Aug 2024 14:49:39 +0000 (09:49 -0500)
committerSteve Myers <steve@notmandatory.org>
Thu, 29 Aug 2024 14:55:21 +0000 (09:55 -0500)
e063ad89bde62e60b10435260e736c66515bf447 fix(esplora+wallet+file_store): remove remaining `println!` usage (Leonardo Lima)
b32b9447e089709b66f24d5700750f0aa6b8d6e1 chore(examples): allow `clippy::print_stdout` in examples (Leonardo Lima)
b614237127ec1e760414fcc8ee7c4b0c77c62d46 fix(tests)!: remove println! usage from tests (Leonardo Lima)
eaa1917a46245e177ad3e65b53e34c681f0043d0 chore: add `print_stdout`/`print_stderr` lints to workspace level (Leonardo Lima)

Pull request description:

  potentially fixes #1362
  <!-- You can erase any parts of this template not applicable to your Pull Request. -->

  ### Description

  It adds both `print_stdout` and `print_stderr` deny level lints on workspace level, but it does allow it on test fns through `clippy.toml` settings, and explicitly allow it on example code.

  <!-- Describe the purpose of this PR, what's being adding and/or fixed -->

  ### Notes to the reviewers
  It currently has the setting allowing it on test fns, but open for discussion below.

  <!-- In this section you can include notes directed to the reviewers, like explaining why some parts
  of the PR were done in a specific way -->

  ### Changelog notice

  - Add both `print_stdout` and `print_stderr` deny level lints on workspace level

  <!-- Notice the release manager should include in the release tag message changelog -->
  <!-- See https://keepachangelog.com/en/1.0.0/ for examples -->

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

  #### New Features:

  * [x] I've added tests for the new feature
  * [x] I've added docs for the new feature

ACKs for top commit:
  notmandatory:
    ACK e063ad89bde62e60b10435260e736c66515bf447

Tree-SHA512: b3348efd86d09944eb36e4d87799eebc23f4e423861b3bad08365d286449f61b29ad332157eecfb7307191ef61d3b8285341e0ccb868581e54b570d6dd37547c

1  2 
Cargo.toml
crates/bitcoind_rpc/Cargo.toml
crates/chain/Cargo.toml
crates/electrum/Cargo.toml
crates/esplora/Cargo.toml
crates/esplora/src/async_ext.rs
crates/esplora/src/blocking_ext.rs
crates/file_store/Cargo.toml
crates/hwi/Cargo.toml
crates/testenv/Cargo.toml
crates/wallet/Cargo.toml

diff --cc Cargo.toml
Simple merge
Simple merge
index ab0670d55c11bdbd715494467e5eb0d2a0141978,0651e718060cbd3f69f78b2a7f7967571960d802..2cc2b44b2c01f36d4f53628e9b6ea6430c72631a
@@@ -12,10 -12,15 +12,13 @@@ readme = "README.md
  
  # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
  
+ [lints]
+ workspace = true
  [dependencies]
  bitcoin = { version = "0.32.0", default-features = false }
 +bdk_core = { path = "../core", version = "0.1", default-features = false }
  serde = { version = "1", optional = true, features = ["derive", "rc"] }
 -
 -# Use hashbrown as a feature flag to have HashSet and HashMap from it.
 -hashbrown = { version = "0.9.1", optional = true, features = ["serde"] }
  miniscript = { version = "12.0.0", optional = true, default-features = false }
  
  # Feature dependencies
index c807cf83052c9cb8bd131b6206102c6981bf6208,489d35a5c497a29aa849c67009cb8574946445f6..f65911f8972c65566df0c4406e9215f0b1d26056
@@@ -9,9 -9,12 +9,12 @@@ description = "Fetch data from electru
  license = "MIT OR Apache-2.0"
  readme = "README.md"
  
+ [lints]
+ workspace = true
  [dependencies]
 -bdk_chain = { path = "../chain", version = "0.17.0" }
 -electrum-client = { version = "0.21", features = ["proxy"], default-features = false }
 +bdk_core = { path = "../core", version = "0.1" }
 +electrum-client = { version = "0.21", features = [ "proxy" ], default-features = false }
  
  [dev-dependencies]
  bdk_testenv = { path = "../testenv", default-features = false }
index d7e5290f323f9a4cd3d0a039f4f3ef8a8711115e,422cf99fb27c7c22a103ec976bfc227f0350cbaf..911c7a6e9138ae354c91ae058c54f16cd89db454
@@@ -11,8 -11,11 +11,11 @@@ readme = "README.md
  
  # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
  
+ [lints]
+ workspace = true
  [dependencies]
 -bdk_chain = { path = "../chain", version = "0.17.0", default-features = false }
 +bdk_core = { path = "../core", version = "0.1", default-features = false }
  esplora-client = { version = "0.9.0", default-features = false }
  async-trait = { version = "0.1.66", optional = true }
  futures = { version = "0.3.26", optional = true }
Simple merge
Simple merge
index 8e8966bb465ac74572eefec5be2c2c3a5da35cd8,50f86c5b8d28db0a757a7bd3fd9cca43b294a6d7..29af6b70f292d1327507d007469ed9998ea487f9
@@@ -10,8 -10,11 +10,11 @@@ keywords = ["bitcoin", "persist", "pers
  authors = ["Bitcoin Dev Kit Developers"]
  readme = "README.md"
  
+ [lints]
+ workspace = true
  [dependencies]
 -bdk_chain = { path = "../chain", version = "0.17.0", features = [ "serde", "miniscript" ] }
 +bdk_chain = { path = "../chain", version = "0.18.0", features = [ "serde", "miniscript" ] }
  bincode = { version = "1" }
  serde = { version = "1", features = ["derive"] }
  
index 947bb835806adfb6e9be1354ecd366fd9ba05e25,154833f1e1ec9b6f9c3215a11581ac8bb15f11b8..fdb1466264eaa55be9e30f8b9012c710e44f697d
@@@ -8,6 -8,9 +8,9 @@@ description = "Utilities to use bdk wit
  license = "MIT OR Apache-2.0"
  readme = "README.md"
  
+ [lints]
+ workspace = true
  [dependencies]
 -bdk_wallet = { path = "../wallet", version = "1.0.0-beta.1" }
 -hwi = { version = "0.9.0", features = [ "miniscript"] }
 +bdk_wallet = { path = "../wallet", version = "1.0.0-beta.2" }
 +hwi = { version = "0.9.0", features = [ "miniscript" ] }
index 688f81fbec297054a10866df80a2e7646bcaa09e,48fa1f6b097228563bb5c5be38ffb2f033d8e6df..c6cef80a24c4c42af3add775601522f4d73884f5
@@@ -12,9 -12,12 +12,12 @@@ readme = "README.md
  
  # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
  
+ [lints]
+ workspace = true
  [dependencies]
 -bdk_chain = { path = "../chain", version = "0.17", default-features = false }
 -electrsd = { version = "0.28.0", features = ["bitcoind_25_0", "esplora_a33e97e1", "legacy"] }
 +bdk_chain = { path = "../chain", version = "0.18", default-features = false }
 +electrsd = { version = "0.28.0", features = [ "bitcoind_25_0", "esplora_a33e97e1", "legacy" ] }
  
  [features]
  default = ["std"]
index afdcf688cdc6fb8dc7a8abf9022c544ba100fae2,aa353170905ad8849727cce837e536cfa871c50b..7e4a16d0fa8e259d97538fb503a6a054a48eaabf
@@@ -12,14 -12,17 +12,17 @@@ authors = ["Bitcoin Dev Kit Developers"
  edition = "2021"
  rust-version = "1.63"
  
+ [lints]
+ workspace = true
  [dependencies]
  rand_core = { version = "0.6.0" }
 -miniscript = { version = "12.0.0", features = ["serde"], default-features = false }
 -bitcoin = { version = "0.32.0", features = ["serde", "base64"], default-features = false }
 +miniscript = { version = "12.0.0", features = [ "serde" ], default-features = false }
 +bitcoin = { version = "0.32.0", features = [ "serde", "base64" ], default-features = false }
  serde = { version = "^1.0", features = ["derive"] }
  serde_json = { version = "^1.0" }
 -bdk_chain = { path = "../chain", version = "0.17.0", features = ["miniscript", "serde"], default-features = false }
 -bdk_file_store = { path = "../file_store", version = "0.14.0", optional = true }
 +bdk_chain = { path = "../chain", version = "0.18.0", features = [ "miniscript", "serde" ], default-features = false }
 +bdk_file_store = { path = "../file_store", version = "0.15.0", optional = true }
  
  # Optional dependencies
  bip39 = { version = "2.0", optional = true }