osmosis-labs / osmosis-rust

Rust libraries for osmosis
Apache License 2.0
59 stars 52 forks source link

Prevent adding unused import "osmosis_std_derive::CosmwasmExt" #103

Open loloicci opened 8 months ago

loloicci commented 8 months ago

Without this PR, transform adds import osmosis_std_derive::CosmwasmExt to all modules given to transform_module to decrease warnings in "osmosis_std".

This PR prevents adding import osmosis_std_derive::CosmwasmExt modules not using it. Modules that do not use import osmosis_std_derive::CosmwasmExt contain only enums and do not contain structs.

note

This PR does not contain executing proto-build. This is because the latest main's (b9ce0b7ec7130e7a6223405416b6c43466e8a9f6) proto-build deletes some attributes and it makes an error in "osmosis-std". Deleted attributes are as follows.

@@ -924,24 +923,3 @@ pub struct Block {
     #[prost(message, optional, tag = "4")]
     pub last_commit: ::core::option::Option<Commit>,
 }
-
-#[allow(clippy::derive_partial_eq_without_eq)]
-#[derive(
-    Clone,
-    PartialEq,
-    Eq,
-    ::prost::Message,
-    ::serde::Serialize,
-    ::serde::Deserialize,
-    ::schemars::JsonSchema,
-    CosmwasmExt,
-)]
-#[proto_message(type_url = "/tendermint.types.EventDataRoundState")]
-pub struct EventDataRoundState {
-    #[prost(int64, tag = "1")]
-    pub height: i64,
-    #[prost(int32, tag = "2")]
-    pub round: i32,
-    #[prost(string, tag = "3")]
-    pub step: String,
-}