Disambiguate between help and height flags by using -H for the latter

guide/src/commands/path-setup/clients.md

@@ -67,7 +67,7 @@ POSITIONAL ARGUMENTS:
     dst_client_id             identifier of the client to be updated on destination chain
 
 FLAGS:
-    -h, --target-height TARGET-HEIGHT
+    -H, --target-height TARGET-HEIGHT
     -t, --trusted-height TRUSTED-HEIGHT
 ```
 

guide/src/commands/queries/channel.md

@@ -79,7 +79,7 @@ POSITIONAL ARGUMENTS:
     channel_id                identifier of the channel to query
 
 FLAGS:
-    -h, --height HEIGHT       height of the state to query
+    -H, --height HEIGHT       height of the state to query
 ```
 
 __Example__
@@ -131,7 +131,7 @@ POSITIONAL ARGUMENTS:
     channel_id                identifier of the channel to query
 
 FLAGS:
-    -h, --height HEIGHT       height of the state to query
+    -H, --height HEIGHT       height of the state to query
     -v, --verbose             enable verbose output, displaying all details of channels, connections & clients
 ```
 
@@ -181,4 +181,4 @@ Success: ChannelEndsSummary {
 ```
 
 Passing the `-v` flag will additionally print all the details of the
-channel, connection, and client on both ends.
+channel, connection, and client on both ends.

guide/src/commands/queries/client.md

@@ -100,7 +100,7 @@ POSITIONAL ARGUMENTS:
     client_id                 identifier of the client to query
 
 FLAGS:
-    -h, --height HEIGHT       the chain height which this query should reflect
+    -H, --height HEIGHT       the chain height which this query should reflect
 ```
 
 __Example__
@@ -159,7 +159,7 @@ POSITIONAL ARGUMENTS:
 FLAGS:
     -c, --consensus-height    CONSENSUS-HEIGHT
     -s, --heights-only        show only consensus heights
-    -h, --height HEIGHT       the chain height context to be used, applicable only to a specific height
+    -H, --height HEIGHT       the chain height context to be used, applicable only to a specific height
 ```
 
 __Example__
@@ -235,7 +235,7 @@ POSITIONAL ARGUMENTS:
     client_id                 identifier of the client to query
 
 FLAGS:
-    -h, --height HEIGHT       the chain height which this query should reflect
+    -H, --height HEIGHT       the chain height which this query should reflect
 ```
 
 __Example__
@@ -268,7 +268,7 @@ POSITIONAL ARGUMENTS:
     consensus_height          height of header to query
 
 FLAGS:
-    -h, --height HEIGHT       the chain height context for the query
+    -H, --height HEIGHT       the chain height context for the query
 ```
 
 __Example__

guide/src/commands/queries/connection.md

@@ -68,7 +68,7 @@ POSITIONAL ARGUMENTS:
     connection_id             identifier of the connection to query
 
 FLAGS:
-    -h, --height HEIGHT       height of the state to query
+    -H, --height HEIGHT       height of the state to query
 ```
 
 __Example__

guide/src/commands/queries/packet.md

@@ -82,7 +82,7 @@ POSITIONAL ARGUMENTS:
     sequence                  sequence of packet to query
 
 FLAGS:
-    -h, --height HEIGHT       height of the state to query
+    -H, --height HEIGHT       height of the state to query
 ```
 
 __Example__
@@ -154,7 +154,7 @@ POSITIONAL ARGUMENTS:
     sequence                  sequence of packet to query
 
 FLAGS:
-    -h, --height HEIGHT       height of the state to query
+    -H, --height HEIGHT       height of the state to query
 ```
 
 __Example__

guide/src/commands/raw/client.md

@@ -63,7 +63,7 @@ POSITIONAL ARGUMENTS:
     dst_client_id             identifier of the client to be updated on destination chain
 
 FLAGS:
-    -h, --target-height TARGET-HEIGHT
+    -H, --target-height TARGET-HEIGHT
     -t, --trusted-height TRUSTED-HEIGHT
 ```
 

relayer-cli/src/commands/query/channel.rs

@@ -24,8 +24,7 @@ pub struct QueryChannelEndCmd {
     #[clap(required = true, about = "identifier of the channel to query")]
     channel_id: ChannelId,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "height of the state to query")]
+    #[clap(short = 'H', long, about = "height of the state to query")]
     height: Option<u64>,
 }
 

relayer-cli/src/commands/query/channel_ends.rs

@@ -27,7 +27,7 @@ pub struct QueryChannelEndsCmd {
     channel_id: ChannelId,
 
     // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "height of the state to query")]
+    #[clap(short = 'H', long, about = "height of the state to query")]
     height: Option<u64>,
 
     #[clap(

relayer-cli/src/commands/query/client.rs

@@ -30,8 +30,7 @@ pub struct QueryClientStateCmd {
     #[clap(required = true, about = "identifier of the client to query")]
     client_id: ClientId,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "the chain height context for the query")]
+    #[clap(short = 'H', long, about = "the chain height context for the query")]
     height: Option<u64>,
 }
 
@@ -84,7 +83,7 @@ pub struct QueryClientConsensusCmd {
     heights_only: bool,
 
     #[clap(
-        short = 'h',
+        short = 'H',
         long,
         about = "the chain height context to be used, applicable only to a specific height"
     )]
@@ -172,8 +171,7 @@ pub struct QueryClientHeaderCmd {
     #[clap(required = true, about = "height of header to query")]
     consensus_height: u64,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "the chain height context for the query")]
+    #[clap(short = 'H', long, about = "the chain height context for the query")]
     height: Option<u64>,
 }
 
@@ -239,7 +237,7 @@ pub struct QueryClientConnectionsCmd {
     client_id: ClientId,
 
     #[clap(
-        short = 'h',
+        short = 'H',
         long,
         about = "the chain height which this query should reflect"
     )]

relayer-cli/src/commands/query/connection.rs

@@ -25,8 +25,7 @@ pub struct QueryConnectionEndCmd {
     #[clap(required = true, about = "identifier of the connection to query")]
     connection_id: ConnectionId,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "height of the state to query")]
+    #[clap(short = 'H', long, about = "height of the state to query")]
     height: Option<u64>,
 }
 

relayer-cli/src/commands/query/packet/ack.rs

@@ -27,8 +27,7 @@ pub struct QueryPacketAcknowledgmentCmd {
     #[clap(required = true, about = "sequence of packet to query")]
     sequence: Sequence,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "height of the state to query")]
+    #[clap(short = 'H', long, about = "height of the state to query")]
     height: Option<u64>,
 }
 

relayer-cli/src/commands/query/packet/commitment.rs

@@ -34,8 +34,7 @@ pub struct QueryPacketCommitmentCmd {
     #[clap(required = true, about = "sequence of packet to query")]
     sequence: Sequence,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "height of the state to query")]
+    #[clap(short = 'H', long, about = "height of the state to query")]
     height: Option<u64>,
 }
 

relayer-cli/src/commands/tx/client.rs

@@ -66,8 +66,7 @@ pub struct TxUpdateClientCmd {
     )]
     dst_client_id: ClientId,
 
-    // FIXME: rename the short option to avoid confusion with --help?
-    #[clap(short = 'h', long, about = "the target height of the client update")]
+    #[clap(short = 'H', long, about = "the target height of the client update")]
     target_height: Option<u64>,
 
     #[clap(short = 't', long, about = "the trusted height of the client update")]