Problem: x/supply is not marked as deprecated (#724)

Solution: Marked `x/supply` grpc and cli as deprecated. Fixes #720.

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## Unreleased
 
 - Upgrade `cosmos-sdk` to `v0.45.1` and `ibc-go` to `v3.0.0-rc0`. #717
+- Mark gRPC and CLI of `x/supply` as deprecated. #724
 
 *March 1, 2022*
 

proto/supply/v1/query.proto

@@ -9,6 +9,8 @@ option go_package = "github.com/crypto-org-chain/chain-main/x/supply/types";
 
 // Query defines the gRPC querier service.
 service Query {
+    option deprecated = true;
+
     // TotalSupply queries the total supply of all coins.
     rpc TotalSupply(SupplyRequest) returns (SupplyResponse) {
         option (google.api.http).get = "/chainmain/supply/v1/total";

x/supply/client/cli/query.go

@@ -17,7 +17,7 @@ import (
 func GetQueryCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:                        types.ModuleName,
-		Short:                      "Querying commands for the supply module",
+		Short:                      "Querying commands for the supply module [Deprecated: do not use]",
 		DisableFlagParsing:         true,
 		SuggestionsMinimumDistance: 2,
 		RunE:                       client.ValidateCmd,
@@ -35,9 +35,9 @@ func GetQueryCmd() *cobra.Command {
 func GetCmdQueryTotalSupply() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "total",
-		Short: "Query the total supply of coins of the chain",
+		Short: "Query the total supply of coins of the chain [Deprecated: do not use]",
 		Long: strings.TrimSpace(
-			fmt.Sprintf(`Query total supply of coins that are held by accounts in the chain.
+			fmt.Sprintf(`Query total supply of coins that are held by accounts in the chain. [Deprecated: do not use]
 Example:
   $ %s query %s total
 `,
@@ -50,6 +50,7 @@ Example:
 				return err
 			}
 
+			// nolint: staticcheck
 			queryClient := types.NewQueryClient(clientCtx)
 			res, err := queryClient.TotalSupply(cmd.Context(), types.NewSupplyRequest())
 			if err != nil {
@@ -69,9 +70,9 @@ Example:
 func GetCmdQueryLiquidSupply() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "liquid",
-		Short: "Query the liquid supply of coins of the chain",
+		Short: "Query the liquid supply of coins of the chain [Deprecated: do not use]",
 		Long: strings.TrimSpace(
-			fmt.Sprintf(`Query liquid supply of coins that are held by accounts in the chain.
+			fmt.Sprintf(`Query liquid supply of coins that are held by accounts in the chain. [Deprecated: do not use]
 Example:
   $ %s query %s liquid
 `,
@@ -84,6 +85,7 @@ Example:
 				return err
 			}
 
+			// nolint: staticcheck
 			queryClient := types.NewQueryClient(clientCtx)
 			res, err := queryClient.LiquidSupply(cmd.Context(), types.NewSupplyRequest())
 			if err != nil {

x/supply/module.go

@@ -80,6 +80,7 @@ func (a AppModuleBasic) RegisterRESTRoutes(clientCtx client.Context, rtr *mux.Ro
 
 // RegisterGRPCGatewayRoutes registers the gRPC Gateway routes for the capability module.
 func (AppModuleBasic) RegisterGRPCGatewayRoutes(clientCtx client.Context, mux *runtime.ServeMux) {
+	// nolint: staticcheck
 	err := types.RegisterQueryHandlerClient(context.Background(), mux, types.NewQueryClient(clientCtx))
 
 	if err != nil {
@@ -134,6 +135,7 @@ func (am AppModule) LegacyQuerierHandler(legacyQuerierCdc *codec.LegacyAmino) sd
 
 // RegisterServices registers query server.
 func (am AppModule) RegisterServices(cfg module.Configurator) {
+	// nolint: staticcheck
 	types.RegisterQueryServer(cfg.QueryServer(), am.keeper)
 }