fix: fix format

Signed-off-by: Esteve Fernandez <esteve@apache.org>

Author: esteve

rclrs/src/action.rs

@@ -9,7 +9,7 @@ pub use action_goal_receiver::*;
 pub(crate) mod action_server;
 pub use action_server::*;
 
-use crate::{log_error, rcl_bindings::*, vendor::builtin_interfaces::msg::Time, DropGuard};
+use crate::{DropGuard, log_error, rcl_bindings::*, vendor::builtin_interfaces::msg::Time};
 use std::fmt;
 
 #[cfg(feature = "serde")]
@@ -31,24 +31,26 @@ impl GoalUuid {
 
 impl fmt::Display for GoalUuid {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> Result<(), fmt::Error> {
-        write!(f, "{:02x}{:02x}{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}{:02x}{:02x}{:02x}{:02x}",
-               self.0[0],
-               self.0[1],
-               self.0[2],
-               self.0[3],
-               self.0[4],
-               self.0[5],
-               self.0[6],
-               self.0[7],
-               self.0[8],
-               self.0[9],
-               self.0[10],
-               self.0[11],
-               self.0[12],
-               self.0[13],
-               self.0[14],
-               self.0[15],
-               )
+        write!(
+            f,
+            "{:02x}{:02x}{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}-{:02x}{:02x}{:02x}{:02x}{:02x}{:02x}",
+            self.0[0],
+            self.0[1],
+            self.0[2],
+            self.0[3],
+            self.0[4],
+            self.0[5],
+            self.0[6],
+            self.0[7],
+            self.0[8],
+            self.0[9],
+            self.0[10],
+            self.0[11],
+            self.0[12],
+            self.0[13],
+            self.0[14],
+            self.0[15],
+        )
     }
 }
 

rclrs/src/action/action_client.rs

@@ -1,12 +1,11 @@
 use super::empty_goal_status_array;
 use crate::{
-    log_warn,
+    CancelResponse, CancelResponseCode, DropGuard, ENTITY_LIFECYCLE_MUTEX, GoalStatus,
+    GoalStatusCode, GoalUuid, MultiCancelResponse, Node, NodeHandle, QoSProfile, RclPrimitive,
+    RclPrimitiveHandle, RclPrimitiveKind, RclrsError, ReadyKind, TakeFailedAsNone, ToResult,
+    Waitable, WaitableLifecycle, log_warn,
     rcl_bindings::*,
     vendor::{action_msgs::srv::CancelGoal_Response, builtin_interfaces::msg::Time},
-    CancelResponse, CancelResponseCode, DropGuard, GoalStatus, GoalStatusCode, GoalUuid,
-    MultiCancelResponse, Node, NodeHandle, QoSProfile, RclPrimitive, RclPrimitiveHandle,
-    RclPrimitiveKind, RclrsError, ReadyKind, TakeFailedAsNone, ToResult, Waitable,
-    WaitableLifecycle, ENTITY_LIFECYCLE_MUTEX,
 };
 use rosidl_runtime_rs::{Action, Message, RmwFeedbackMessage, RmwGoalResponse, RmwResultResponse};
 use std::{
@@ -17,8 +16,8 @@ use std::{
     sync::{Arc, Mutex, MutexGuard, Weak},
 };
 use tokio::sync::{
-    mpsc::{unbounded_channel, UnboundedSender},
-    oneshot::{channel as oneshot_channel, Sender},
+    mpsc::{UnboundedSender, unbounded_channel},
+    oneshot::{Sender, channel as oneshot_channel},
     watch::Sender as WatchSender,
 };
 

rclrs/src/action/action_client/goal_client.rs

@@ -1,6 +1,6 @@
 use crate::{
-    vendor::builtin_interfaces::msg::Time, CancellationClient, FeedbackClient, GoalStatus,
-    GoalStatusCode, ResultClient, StatusWatcher,
+    CancellationClient, FeedbackClient, GoalStatus, GoalStatusCode, ResultClient, StatusWatcher,
+    vendor::builtin_interfaces::msg::Time,
 };
 use rosidl_runtime_rs::Action;
 use std::{

rclrs/src/action/action_client/result_client.rs

@@ -1,6 +1,6 @@
 use super::GoalClientLifecycle;
 use crate::GoalStatusCode;
-use futures::{future::Shared, FutureExt};
+use futures::{FutureExt, future::Shared};
 use rosidl_runtime_rs::Action;
 use std::{
     future::Future,

rclrs/src/action/action_goal_receiver.rs

@@ -8,7 +8,7 @@ use std::{
     ops::{Deref, DerefMut},
     sync::Arc,
 };
-use tokio::sync::mpsc::{unbounded_channel, UnboundedReceiver};
+use tokio::sync::mpsc::{UnboundedReceiver, unbounded_channel};
 
 /// This is an alternative tool for implementing an [`ActionServer`]. Instead of
 /// specifying a callback to receive goal requests, you can use this receiver to

rclrs/src/action/action_server.rs

@@ -1,10 +1,9 @@
 use super::empty_goal_status_array;
 use crate::{
-    action::GoalUuid, error::ToResult, rcl_bindings::*,
-    vendor::action_msgs::srv::CancelGoal_Response, ActionGoalReceiver, CancelResponseCode,
-    DropGuard, GoalStatusCode, Node, NodeHandle, QoSProfile, RclPrimitive, RclPrimitiveHandle,
-    RclPrimitiveKind, RclrsError, ReadyKind, TakeFailedAsNone, Waitable, WaitableLifecycle,
-    ENTITY_LIFECYCLE_MUTEX,
+    ActionGoalReceiver, CancelResponseCode, DropGuard, ENTITY_LIFECYCLE_MUTEX, GoalStatusCode,
+    Node, NodeHandle, QoSProfile, RclPrimitive, RclPrimitiveHandle, RclPrimitiveKind, RclrsError,
+    ReadyKind, TakeFailedAsNone, Waitable, WaitableLifecycle, action::GoalUuid, error::ToResult,
+    rcl_bindings::*, vendor::action_msgs::srv::CancelGoal_Response,
 };
 use futures::future::BoxFuture;
 use rosidl_runtime_rs::{Action, Message, RmwGoalRequest, RmwResultRequest};

rclrs/src/action/action_server/action_server_goal_handle.rs

@@ -1,7 +1,7 @@
 use super::GoalStatusCode;
 use crate::{
-    log_error, rcl_bindings::*, ActionServerHandle, GoalUuid, RclErrorMsg, RclReturnCode,
-    RclrsError, ToResult,
+    ActionServerHandle, GoalUuid, RclErrorMsg, RclReturnCode, RclrsError, ToResult, log_error,
+    rcl_bindings::*,
 };
 use rosidl_runtime_rs::{Action, RmwResultResponse};
 use std::sync::{Mutex, MutexGuard};

rclrs/src/action/action_server/cancellation_state.rs

@@ -1,15 +1,14 @@
 use super::ActionServerHandle;
 use crate::{
-    log_error,
+    CancelResponseCode, GoalUuid, Node, RclrsErrorFilter, ToResult, log_error,
     rcl_bindings::*,
     vendor::{
         action_msgs::{msg::GoalInfo, srv::CancelGoal_Response},
         unique_identifier_msgs::msg::UUID,
     },
-    CancelResponseCode, GoalUuid, Node, RclrsErrorFilter, ToResult,
 };
 use futures::{
-    future::{select, Either},
+    future::{Either, select},
     pin_mut,
 };
 use futures_lite::future::race;
@@ -20,7 +19,7 @@ use std::{
     future::Future,
     sync::{Arc, Mutex},
 };
-use tokio::sync::watch::{channel as watch_channel, Receiver, Sender};
+use tokio::sync::watch::{Receiver, Sender, channel as watch_channel};
 
 pub(super) struct CancellationState<A: Action> {
     receiver: Receiver<bool>,

rclrs/src/action/action_server/live_action_server_goal.rs

@@ -2,7 +2,7 @@ use super::{
     ActionServerGoalHandle, ActionServerHandle, CancellationRequest, CancellationState,
     GoalStatusCode, TerminalStatus,
 };
-use crate::{log_error, rcl_bindings::*, RclrsError, ToResult};
+use crate::{RclrsError, ToResult, log_error, rcl_bindings::*};
 use rosidl_runtime_rs::{Action, Message};
 use std::{borrow::Cow, ops::Deref, sync::Arc};
 

rclrs/src/action/action_server/requested_goal.rs

@@ -2,7 +2,7 @@ use super::{
     AcceptedGoal, ActionServerGoalBoard, ActionServerGoalHandle, LiveActionServerGoal,
     TerminatedGoal,
 };
-use crate::{log_error, rcl_bindings::*, GoalUuid, RclrsError, RclrsErrorFilter, ToResult};
+use crate::{GoalUuid, RclrsError, RclrsErrorFilter, ToResult, log_error, rcl_bindings::*};
 use rosidl_runtime_rs::Action;
 use std::sync::Arc;