Moved silencers to the bottom of imports

mirrord/agent/src/main.rs

@@ -7,6 +7,12 @@
 #![warn(clippy::indexing_slicing)]
 #![deny(unused_crate_dependencies)]
 
+/// Silences `deny(unused_crate_dependencies)`.
+///
+/// This dependency is only used in integration tests.
+#[cfg(test)]
+use test_bin as _;
+
 mod cli;
 mod client_connection;
 mod container_handle;
@@ -29,9 +35,3 @@ mod watched_task;
 async fn main() -> crate::error::Result<()> {
     crate::entrypoint::main().await
 }
-
-/// Silences `deny(unused_crate_dependencies)`.
-///
-/// This dependency is only used in integration tests.
-#[cfg(test)]
-use test_bin as _;

mirrord/auth/src/lib.rs

@@ -4,6 +4,12 @@
 pub use pem;
 pub use x509_certificate;
 
+/// Silences `deny(unused_crate_dependencies)`.
+/// Although we don't use this dependency directly,
+/// compilation fails without it.
+#[cfg(feature = "client")]
+use k8s_openapi as _;
+
 /// X509 Certificate abstraction for serialization and deserialization
 pub mod certificate;
 /// FileSystem based storage for multiple credentials (default contents "~/.mirrord/credentials")
@@ -16,9 +22,3 @@ pub mod credentials;
 pub mod error;
 /// Public/Private key abstraction for serialization and deserialization
 pub mod key_pair;
-
-/// Silences `deny(unused_crate_dependencies)`.
-/// Although we don't use this dependency directly,
-/// compilation fails without it.
-#[cfg(feature = "client")]
-use k8s_openapi as _;

mirrord/console/src/lib.rs

@@ -1,16 +1,6 @@
 #![warn(clippy::indexing_slicing)]
 #![deny(unused_crate_dependencies)]
 
-#[cfg(feature = "async-logger")]
-pub mod async_logger;
-pub mod error;
-pub mod logger;
-pub mod protocol;
-
-#[cfg(feature = "async-logger")]
-pub use async_logger::init_async_logger;
-pub use logger::init_logger;
-
 /// Silences `deny(unused_crate_dependencies)`.
 ///
 /// These dependencies are only used in the console binary.
@@ -20,3 +10,13 @@ mod binary_deps {
     use tracing as _;
     use tracing_subscriber as _;
 }
+
+#[cfg(feature = "async-logger")]
+pub mod async_logger;
+pub mod error;
+pub mod logger;
+pub mod protocol;
+
+#[cfg(feature = "async-logger")]
+pub use async_logger::init_async_logger;
+pub use logger::init_logger;

mirrord/layer/src/lib.rs

@@ -101,6 +101,20 @@ use crate::{
     load::LoadType,
 };
 
+/// Silences `deny(unused_crate_dependencies)`.
+///
+/// These dependencies are only used in integration tests.
+#[cfg(test)]
+mod integration_tests_deps {
+    use actix_codec as _;
+    use futures as _;
+    use mirrord_intproxy as _;
+    use tempfile as _;
+    use test_cdylib as _;
+    use tests as _;
+    use tokio as _;
+}
+
 mod common;
 mod debugger_ports;
 mod detour;
@@ -712,17 +726,3 @@ pub(crate) unsafe extern "C" fn uv_fs_close_detour(
     close_layer_fd(fd);
     FN_UV_FS_CLOSE(a, b, fd, c)
 }
-
-/// Silences `deny(unused_crate_dependencies)`.
-///
-/// These dependencies are only used in integration tests.
-#[cfg(test)]
-mod integration_tests_deps {
-    use actix_codec as _;
-    use futures as _;
-    use mirrord_intproxy as _;
-    use tempfile as _;
-    use test_cdylib as _;
-    use tests as _;
-    use tokio as _;
-}