Skip to content

Commit

Permalink
chore: Migrate print-ban from test to clippy
Browse files Browse the repository at this point in the history
This should be more resilient to false positives like in rust-lang#12245 where a
string contains `println`.
  • Loading branch information
epage committed Jun 9, 2023
1 parent ccd2540 commit 9df5e79
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 107 deletions.
2 changes: 2 additions & 0 deletions clippy.toml
@@ -1,3 +1,5 @@
allow-print-in-tests = true
allow-dbg-in-tests = true
disallowed-methods = [
{ path = "std::env::var", reason = "Use `Config::get_env` instead. See rust-lang/cargo#11588" },
{ path = "std::env::var_os", reason = "Use `Config::get_env_os` instead. See rust-lang/cargo#11588" },
Expand Down
3 changes: 3 additions & 0 deletions src/cargo/lib.rs
Expand Up @@ -6,6 +6,9 @@
#![allow(clippy::all)]
#![warn(clippy::disallowed_methods)]
#![warn(clippy::self_named_module_files)]
#![warn(clippy::print_stdout)]
#![warn(clippy::print_stderr)]
#![warn(clippy::dbg_macro)]
#![allow(rustdoc::private_intra_doc_links)]

//! # Cargo as a library
Expand Down
107 changes: 0 additions & 107 deletions tests/internal.rs

This file was deleted.

0 comments on commit 9df5e79

Please sign in to comment.