Add SilenceUsage flag to hide menu when an error occurs.

[pinosu]

Resolves #973

[codecov]

Codecov Report

Merging #1068 (16170cf) into main (84e24d3) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1068      +/-   ##
==========================================
- Coverage   59.28%   59.16%   -0.12%     
==========================================
  Files          53       53              
  Lines        6722     6735      +13     
==========================================
  Hits         3985     3985              
- Misses       2442     2455      +13     
  Partials      295      295              

Impacted Files	Coverage Δ
x/wasm/client/cli/genesis_msg.go	71.42% <ø> (ø)
x/wasm/client/cli/gov_tx.go	6.52% <ø> (ø)
x/wasm/client/cli/new_tx.go	0.00% <ø> (ø)
x/wasm/client/cli/query.go	0.00% <0.00%> (ø)
x/wasm/client/cli/tx.go	32.50% <0.00%> (-0.14%)	⬇️
x/wasm/module.go	37.90% <0.00%> (ø)
x/wasm/types/proposal.go	61.31% <0.00%> (-1.81%)	⬇️

[alpe]

Thanks for the PR!

The changes look good but a lot of commands were touched. I was wondering if it is sufficient to set SilenceUsage: true on the GetTxCmd() only. WDYT?

[pinosu]

Thanks for the PR!

The changes look good but a lot of commands were touched. I was wondering if it is sufficient to set SilenceUsage: true on the GetTxCmd() only. WDYT?

I agree that a more general soluton would be better. Unfortunately, I have tried your suggestion as first option but it didn't work. I had to add it to every command.

[alpe]

👍 Good work