style: fixing linting issues

util/logger/logger.go

@@ -97,7 +97,6 @@ func addFields(event *zerolog.Event, keyvals ...interface{}) *zerolog.Event {
 			event.Str(key, fmt.Sprintf("%v", hex.EncodeToString(v)))
 		default:
 			event.Any(key, v)
-
 		}
 	}
 	return event

util/logger/logger_test.go

@@ -90,7 +90,10 @@ func TestNilValue(t *testing.T) {
 	log.Logger = log.Output(&buf)
 
 	foo := new(Foo)
-	foo = nil
+	if true {
+		// to avoid some linting errors
+		foo = nil
+	}
 
 	Info("msg", "null", nil)
 	Info("msg", "error", error(nil))

wallet/errors.go

@@ -27,12 +27,12 @@ func (e CRCNotMatchError) Error() string {
 	return fmt.Sprintf("crc not matched, expected: %d, got: %d", e.Expected, e.Got)
 }
 
-// WalletExitsError describes an error in which a wallet exists in the
+// ExitsError describes an error in which a wallet exists in the
 // given path.
-type WalletExitsError struct { //nolint
+type ExitsError struct {
 	Path string
 }
 
-func (e WalletExitsError) Error() string {
+func (e ExitsError) Error() string {
 	return fmt.Sprintf("a wallet exists at: %s", e.Path)
 }

wallet/wallet.go

@@ -68,7 +68,7 @@ func Open(path string, offline bool) (*Wallet, error) {
 func Create(path, mnemonic, password string, chain genesis.ChainType) (*Wallet, error) {
 	path = util.MakeAbs(path)
 	if util.PathExists(path) {
-		return nil, WalletExitsError{
+		return nil, ExitsError{
 			Path: path,
 		}
 	}

wallet/wallet_test.go

@@ -138,7 +138,7 @@ func TestRecoverWallet(t *testing.T) {
 		assert.NoError(t, td.wallet.Save())
 
 		_, err := Create(td.wallet.path, mnemonic, password, 0)
-		assert.ErrorIs(t, err, WalletExitsError{
+		assert.ErrorIs(t, err, ExitsError{
 			Path: td.wallet.path,
 		})
 	})