Linting for Tutorial (#134)

* litning

* linting fix

Author: tac0turtle

.golangci.yml

@@ -1,21 +1,7 @@
 linters:
-  disable-all: true
-  enable:
-    - errcheck
+  enable-all: true
+  disable:
     - golint
-    - ineffassign
-    - unconvert
-    - misspell
-    - govet
-linters-settings:
-  gocyclo:
-    min-complexity: 11
-  errcheck:
-    ignore: fmt:.*,io/ioutil:^Read.*,github.com/spf13/cobra:MarkFlagRequired,github.com/spf13/viper:BindPFlag
-  golint:
-    min-confidence: 1.1
-issues:
-  exclude:
-    - composite
-run:
-  tests: false
+    - unparam
+    - gochecknoinits
+    - gochecknoglobals

app.go

@@ -54,8 +54,8 @@ var (
 	maccPerms = map[string][]string{
 		auth.FeeCollectorName:     nil,
 		distr.ModuleName:          nil,
-		staking.BondedPoolName:    []string{supply.Burner, supply.Staking},
-		staking.NotBondedPoolName: []string{supply.Burner, supply.Staking},
+		staking.BondedPoolName:    {supply.Burner, supply.Staking},
+		staking.NotBondedPoolName: {supply.Burner, supply.Staking},
 	}
 )
 
@@ -157,7 +157,8 @@ func NewNameServiceApp(logger log.Logger, db dbm.DB) *nameServiceApp {
 		app.accountKeeper,
 		app.bankKeeper,
 		supply.DefaultCodespace,
-		maccPerms)
+		maccPerms,
+	)
 
 	// The staking keeper
 	stakingKeeper := staking.NewKeeper(

cmd/nscli/main.go

@@ -18,11 +18,6 @@ import (
 	"github.com/tendermint/tendermint/libs/cli"
 )
 
-const (
-	storeAcc = "acc"
-	storeNS  = "nameservice"
-)
-
 func main() {
 	cobra.EnableCommandSorting = false
 

cmd/nsd/main.go

@@ -43,7 +43,10 @@ func main() {
 	rootCmd.AddCommand(
 		genutilcli.InitCmd(ctx, cdc, app.ModuleBasics, app.DefaultNodeHome),
 		genutilcli.CollectGenTxsCmd(ctx, cdc, genaccounts.AppModuleBasic{}, app.DefaultNodeHome),
-		genutilcli.GenTxCmd(ctx, cdc, app.ModuleBasics, staking.AppModuleBasic{}, genaccounts.AppModuleBasic{}, app.DefaultNodeHome, app.DefaultCLIHome),
+		genutilcli.GenTxCmd(
+			ctx, cdc, app.ModuleBasics, staking.AppModuleBasic{},
+			genaccounts.AppModuleBasic{}, app.DefaultNodeHome, app.DefaultCLIHome,
+		),
 		genutilcli.ValidateGenesisCmd(ctx, cdc, app.ModuleBasics),
 		// AddGenesisAccountCmd allows users to add accounts to the genesis file
 		genaccscli.AddGenesisAccountCmd(ctx, cdc, app.DefaultNodeHome, app.DefaultCLIHome),

tutorial/cn/13-entrypoint.md

@@ -1,8 +1,8 @@
 # Entrypoint
 
-golang的规范是把编译成可执行程序的文件放在项目的`./cmd`文件夹中。对于你的应用程序，您要创建2个可执行程序：
+golang 的规范是把编译成可执行程序的文件放在项目的`./cmd`文件夹中。对于你的应用程序，您要创建 2 个可执行程序：
 
-- `nsd` : 此可执行程序类似于`bitcoind`或其他加密货币的daemon，因为它维护p2p连接，广播交易，处理本地存储并提供用以与网络交互的RPC接口。在这种情况下，Tendermint被用于网络层和排序交易。
+- `nsd` : 此可执行程序类似于`bitcoind`或其他加密货币的 daemon，因为它维护 p2p 连接，广播交易，处理本地存储并提供用以与网络交互的 RPC 接口。在这种情况下，Tendermint 被用于网络层和排序交易。
 - `nscli` : 此可执行程序提供用户与你的应用程序交互的命令。
 
 首先请在项目目录中创建两个将会被实例化成这可执行程序的文件：
@@ -251,16 +251,16 @@ func SimpleAppGenTx(cdc *codec.Codec, pk crypto.PubKey) (
 
 注意上述代码中：
 
-- 上面的大部分代码都结合了来自以下包的CLI命令：
+- 上面的大部分代码都结合了来自以下包的 CLI 命令：
   1. Tendermint
   2. Cosmos-SDK
-  3. 你的nameservice模块
+  3. 你的 nameservice 模块
 - `InitCmd`允许应用程序从配置中生成创世纪状态。深入了解函数调用，以了解有关区块链初始化过程的更多信息。
 - `AddGenesisAccountCmd`可以方便地将帐户添加到创世文件中，允许在区块链启动时就使用资产钱包。
 
 ## nscli
 
-通过构建nscli命令完成：
+通过构建 nscli 命令完成：
 
 > 注意：你的应用程序需要导入你刚编写的代码。这里导入路径设置为此存储库（`github.com/cosmos/sdk-application-tutorial`）。如果您是在自己的仓库中进行的前面的操作，则需要更改导入路径（github.com/{.Username}/{.Project.Repo}）。
 
@@ -291,11 +291,6 @@ import (
 	nsrest "github.com/cosmos/sdk-application-tutorial/x/nameservice/client/rest"
 )
 
-const (
-	storeAcc = "acc"
-	storeNS  = "nameservice"
-)
-
 var defaultCLIHome = os.ExpandEnv("$HOME/.nscli")
 
 func main() {
@@ -336,7 +331,7 @@ func main() {
 		client.LineBreak,
 		keys.Commands(),
 		client.LineBreak,
-	
+
 	)
 
 	executor := cli.PrepareMainCmd(rootCmd, "NS", defaultCLIHome)
@@ -425,10 +420,9 @@ func initConfig(cmd *cobra.Command) error {
 
 注意：
 
-- 代码结合了来自以下包的CLI命令：Tendermint、Cosmos-SDK、你的nameservice模块。
-- [`cobra` CLI文档](https://github.com/spf13/cobra)将有助于理解上述代码。
+- 代码结合了来自以下包的 CLI 命令：Tendermint、Cosmos-SDK、你的 nameservice 模块。
+- [`cobra` CLI 文档](https://github.com/spf13/cobra)将有助于理解上述代码。
 - 你可以在这里看到之前定义的`ModuleClient`。
 - 注意如何将路由包含在`registerRoutes`函数中
 
 ### 现在你已经定义了二进制文件，那么就可以来处理[依赖关系管理并构建应用程序](./14-dep.md)！
-

tutorial/entrypoint.md

@@ -132,10 +132,6 @@ import (
 	"github.com/tendermint/tendermint/libs/cli"
 )
 
-const (
-	storeAcc = "acc"
-	storeNS  = "nameservice"
-)
 
 func main() {
 	cobra.EnableCommandSorting = false

x/nameservice/alias.go

@@ -15,8 +15,8 @@ var (
 	NewMsgSetName    = types.NewMsgSetName
 	NewMsgDeleteName = types.NewMsgDeleteName
 	NewWhois         = types.NewWhois
-	ModuleCdc     	 = types.ModuleCdc
-	RegisterCodec 	 = types.RegisterCodec
+	ModuleCdc        = types.ModuleCdc
+	RegisterCodec    = types.RegisterCodec
 )
 
 type (

x/nameservice/genesis.go

@@ -18,13 +18,13 @@ func NewGenesisState(whoIsRecords []Whois) GenesisState {
 func ValidateGenesis(data GenesisState) error {
 	for _, record := range data.WhoisRecords {
 		if record.Owner == nil {
-			return fmt.Errorf("Invalid WhoisRecord: Value: %s. Error: Missing Owner", record.Value)
+			return fmt.Errorf("invalid WhoisRecord: Value: %s. Error: Missing Owner", record.Value)
 		}
 		if record.Value == "" {
-			return fmt.Errorf("Invalid WhoisRecord: Owner: %s. Error: Missing Value", record.Owner)
+			return fmt.Errorf("invalid WhoisRecord: Owner: %s. Error: Missing Value", record.Owner)
 		}
 		if record.Price == nil {
-			return fmt.Errorf("Invalid WhoisRecord: Value: %s. Error: Missing Price", record.Value)
+			return fmt.Errorf("invalid WhoisRecord: Value: %s. Error: Missing Price", record.Value)
 		}
 	}
 	return nil
@@ -47,10 +47,11 @@ func ExportGenesis(ctx sdk.Context, k Keeper) GenesisState {
 	var records []Whois
 	iterator := k.GetNamesIterator(ctx)
 	for ; iterator.Valid(); iterator.Next() {
+
 		name := string(iterator.Key())
-		var whois Whois
-		whois = k.GetWhois(ctx, name)
+		whois := k.GetWhois(ctx, name)
 		records = append(records, whois)
+
 	}
 	return GenesisState{WhoisRecords: records}
 }

x/nameservice/handler.go

@@ -2,6 +2,7 @@ package nameservice
 
 import (
 	"fmt"
+
 	"github.com/cosmos/sdk-application-tutorial/x/nameservice/types"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
@@ -35,7 +36,8 @@ func handleMsgSetName(ctx sdk.Context, keeper Keeper, msg MsgSetName) sdk.Result
 
 // Handle a message to buy name
 func handleMsgBuyName(ctx sdk.Context, keeper Keeper, msg MsgBuyName) sdk.Result {
-	if keeper.GetPrice(ctx, msg.Name).IsAllGT(msg.Bid) { // Checks if the the bid price is greater than the price paid by the current owner
+	// Checks if the the bid price is greater than the price paid by the current owner
+	if keeper.GetPrice(ctx, msg.Name).IsAllGT(msg.Bid) {
 		return sdk.ErrInsufficientCoins("Bid not high enough").Result() // If not, throw an error
 	}
 	if keeper.HasOwner(ctx, msg.Name) {

x/nameservice/keeper.go

@@ -7,7 +7,7 @@ import (
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
 
-// Keeper maintains the link to data storage and exposes getter/setter methods for the various parts of the state machine
+// Keeper maintains the link to storage and exposes getter/setter methods for the various parts of the state machine
 type Keeper struct {
 	coinKeeper bank.Keeper
 
@@ -104,4 +104,3 @@ func (k Keeper) IsNamePresent(ctx sdk.Context, name string) bool {
 	store := ctx.KVStore(k.storeKey)
 	return store.Has([]byte(name))
 }
-