Auto merge of #12246 - epage:lint, r=weihanglo

chore: Migrate print-ban from test to clippy

This should be more resilient to false positives like in #12245 where a string contains `println`.

See also #12224

.github/workflows/main.yml

@@ -218,6 +218,7 @@ jobs:
     name: bors build finished
     needs:
     - build_std
+    - clippy
     - docs
     - lockfile
     - resolver
@@ -234,6 +235,7 @@ jobs:
     name: bors build finished
     needs:
     - build_std
+    - clippy
     - docs
     - lockfile
     - resolver

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" },

src/cargo/lib.rs

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