Chore: fix feature specific lint warnings

- Fix: databendlabs#1256

examples/raft-kv-memstore/tests/cluster/test_cluster.rs

@@ -1,7 +1,6 @@
 use std::backtrace::Backtrace;
 use std::collections::BTreeMap;
-#[allow(deprecated)] // since nightly 1.82
-use std::panic::PanicInfo;
+use std::panic::PanicHookInfo;
 use std::thread;
 use std::time::Duration;
 
@@ -14,8 +13,7 @@ use raft_kv_memstore::store::Request;
 use tokio::runtime::Runtime;
 use tracing_subscriber::EnvFilter;
 
-#[allow(deprecated)] // PanicInfo deprecated since nightly 1.82
-pub fn log_panic(panic: &PanicInfo) {
+pub fn log_panic(panic: &PanicHookInfo) {
     let backtrace = {
         format!("{:?}", Backtrace::force_capture())
         // #[cfg(feature = "bt")]

openraft/src/core/tick.rs

@@ -156,6 +156,10 @@ where C: RaftTypeConfig
     }
 }
 
+// AsyncRuntime::spawn is `spawn_local` with singlethreaded enabled.
+// It will result in a panic:
+// `spawn_local` called from outside of a `task::LocalSet`.
+#[cfg(not(feature = "singlethreaded"))]
 #[cfg(test)]
 mod tests {
     use std::io::Cursor;
@@ -181,10 +185,6 @@ mod tests {
         type Responder = crate::impls::OneshotResponder<Self>;
     }
 
-    // AsyncRuntime::spawn is `spawn_local` with singlethreaded enabled.
-    // It will result in a panic:
-    // `spawn_local` called from outside of a `task::LocalSet`.
-    #[cfg(not(feature = "singlethreaded"))]
     #[tokio::test]
     async fn test_shutdown() -> anyhow::Result<()> {
         let (tx, mut rx) = TickUTConfig::mpsc_unbounded();

openraft/src/network/snapshot_transport.rs

@@ -314,7 +314,7 @@ pub trait SnapshotTransport<C: RaftTypeConfig> {
     ///
     /// - The receiving state `streaming` is maintained by the caller.
     /// - And it depends on `Raft::begin_receiving_snapshot()` to create a `SnapshotData` for
-    /// receiving data.
+    ///   receiving data.
     ///
     /// Example usage:
     /// ```ignore

openraft/src/network/v2/network.rs

@@ -92,9 +92,9 @@ where C: RaftTypeConfig
     /// [`Raft::handle_transfer_leader()`]: crate::raft::Raft::handle_transfer_leader
     #[since(version = "0.10.0")]
     async fn transfer_leader(&mut self, _req: TransferLeaderRequest<C>, _option: RPCOption) -> Result<(), RPCError<C>> {
-        return Err(RPCError::Unreachable(Unreachable::new(&AnyError::error(
+        Err(RPCError::Unreachable(Unreachable::new(&AnyError::error(
             "transfer_leader not implemented",
-        ))));
+        ))))
     }
 
     /// Build a backoff instance if the target node is temporarily(or permanently) unreachable.