]> Untitled Git - bdk/commit
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)
commita8d52e658bd45eacca683b36d516950b40bb73e5
tree5e5eb501b0ef88e55c298d4e059002c5ff7034d0
parent775e4aee3a9ec2d99adb129f7a9367a3a15c7032
parente063ad89bde62e60b10435260e736c66515bf447
Merge bitcoindevkit/bdk#1425: chore: add `print_stdout`/`print_stderr` lints to workspace level

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
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