Fix clippy on generated code & add a deny for warnings (#640)

* fix clippy on generated code

* remove random comment

* fix clippy cmd

Dockerfile

@@ -9,7 +9,7 @@ ENV RUSTUP_PERMIT_COPY_RENAME "yes"
 
 RUN cargo check
 RUN cargo fmt --check
-RUN cargo clippy --all-features --no-deps
+RUN cargo clippy --all-features --no-deps -- -Dwarnings
 # some tests are setup as integration tests 👀 xmtp_mls
 RUN for crate in xmtp_cryptography xmtp_proto xmtp_v2; do cd ${crate}; cargo test; done
 

xmtp_mls/src/groups/subscriptions.rs

@@ -75,7 +75,7 @@ where
         client: Arc<Client<ApiClient>>,
         group_id: Vec<u8>,
         created_at_ns: i64,
-        mut callback: impl FnMut(StoredGroupMessage) + Send + 'static,
+        callback: impl FnMut(StoredGroupMessage) + Send + 'static,
     ) -> Result<StreamCloser, GroupError> {
         Ok(Client::<ApiClient>::stream_messages_with_callback(
             client,
@@ -86,7 +86,7 @@ where
                     cursor: 0,
                 },
             )]),
-            move |message| callback(message),
+            callback,
         )?)
     }
 }

xmtp_proto/src/lib.rs

@@ -1,3 +1,8 @@
-include!("gen/mod.rs");
+#[allow(clippy::all)]
+mod generated {
+    include!("gen/mod.rs");
+}
+pub use generated::*;
+
 #[cfg(feature = "xmtp-message_api-v1")]
 pub mod api_client;