fix: dont take owned self in Client.stop

Author: obmarg

src/client/mod.rs

@@ -126,7 +126,7 @@ impl Client {
     /// # Errors
     ///
     /// Will return `Err` if the connection actor has already been shut down.
-    pub async fn stop(self, subscription_id: SubscriptionId) -> Result<(), Error> {
+    pub async fn stop(&self, subscription_id: SubscriptionId) -> Result<(), Error> {
         self.actor
             .send(ConnectionCommand::Cancel(subscription_id))
             .await

src/client/subscription.rs

@@ -41,7 +41,7 @@ impl<Operation> Subscription<Operation>
 where
     Operation: GraphqlOperation + Send,
 {
-    /// Returns the identifier for this subscription. 
+    /// Returns the identifier for this subscription.
     ///
     /// This can be used with [`crate::Client::stop`] to stop
     /// a running subscription without needing access to the `Subscription`