lint

axelarnetwork · Sep 14, 2024 · 1329db4 · 1329db4
1 parent 8cdd8d7
commit 1329db4
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 11 deletions.
diff --git a/vald/multisig/mgr.go b/vald/multisig/mgr.go
@@ -2,6 +2,7 @@ package multisig
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"time"
 
@@ -55,7 +56,7 @@ func (mgr Mgr) generateKey(keyUID string) (exported.PublicKey, error) {
 	case *tofnd.KeygenResponse_PubKey:
 		return res.GetPubKey(), nil
 	case *tofnd.KeygenResponse_Error:
-		return nil, fmt.Errorf(res.GetError())
+		return nil, errors.New(res.GetError())
 	default:
 		panic(fmt.Errorf("unknown multisig keygen response %T", res.GetKeygenResponse()))
 	}
@@ -72,14 +73,14 @@ func (mgr Mgr) sign(keyUID string, payloadHash exported.Hash, pubKey []byte) (ty
 		PubKey:    pubKey,
 	})
 	if err != nil {
-		return nil, sdkerrors.Wrapf(err, "failed signing")
+		return nil, sdkerrors.Wrap(err, "failed signing")
 	}
 
 	switch res.GetSignResponse().(type) {
 	case *tofnd.SignResponse_Signature:
 		return res.GetSignature(), nil
 	case *tofnd.SignResponse_Error:
-		return nil, fmt.Errorf(res.GetError())
+		return nil, errors.New(res.GetError())
 	default:
 		panic(fmt.Errorf("unknown multisig sign response %T", res.GetSignResponse()))
 	}

diff --git a/vald/sign.go b/vald/sign.go
@@ -13,6 +13,7 @@ import (
 	sdk "github.com/cosmos/cosmos-sdk/types"
 	sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
 	"github.com/ethereum/go-ethereum/common"
+	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 
 	"github.com/axelarnetwork/axelar-core/utils"
@@ -126,7 +127,7 @@ func GetSignCommand() *cobra.Command {
 
 				return nil
 			case *tofnd.SignResponse_Error:
-				return fmt.Errorf(res.GetError())
+				return errors.New(res.GetError())
 			default:
 				panic(fmt.Errorf("unknown multisig sign response %T", res.GetSignResponse()))
 			}

diff --git a/x/ante/check_proxy.go b/x/ante/check_proxy.go
@@ -32,7 +32,7 @@ func (d CheckProxy) AnteHandle(ctx sdk.Context, msgs []sdk.Msg, simulate bool, n
 		case *stakingtypes.MsgCreateValidator:
 			valAddress, err := sdk.ValAddressFromBech32(msg.ValidatorAddress)
 			if err != nil {
-				return ctx, sdkerrors.Wrapf(sdkerrors.ErrInvalidRequest, err.Error())
+				return ctx, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, err.Error())
 			}
 			if proxy, active := d.snapshotter.GetProxy(ctx, valAddress); proxy.Empty() || !active {
 				return ctx, sdkerrors.Wrapf(sdkerrors.ErrInvalidRequest, "no proxy found for operator %s", valAddress.String())

diff --git a/x/ante/check_refund.go b/x/ante/check_refund.go
@@ -67,7 +67,7 @@ func (d CheckRefundFeeDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, simulate
 				req := *msg
 				err := d.reward.SetPendingRefund(ctx, req, rewardtypes.Refund{Payer: feePayer, Fees: splitFees[0]})
 				if err != nil {
-					return ctx, sdkerrors.Wrapf(sdkerrors.ErrInvalidRequest, err.Error())
+					return ctx, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, err.Error())
 				}
 				// when messages are batched not all are refundable, so we cannot use the msg index directly
 				splitFees = splitFees[1:]

diff --git a/x/ante/undelegate.go b/x/ante/undelegate.go
@@ -32,12 +32,12 @@ func (d UndelegateDecorator) AnteHandle(ctx sdk.Context, msgs []sdk.Msg, simulat
 		case *stakingtypes.MsgUndelegate:
 			valAddress, err := sdk.ValAddressFromBech32(msg.ValidatorAddress)
 			if err != nil {
-				return ctx, sdkerrors.Wrapf(sdkerrors.ErrInvalidRequest, err.Error())
+				return ctx, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, err.Error())
 			}
 
 			delegatorAddress, err := sdk.AccAddressFromBech32(msg.DelegatorAddress)
 			if err != nil {
-				return ctx, sdkerrors.Wrapf(sdkerrors.ErrInvalidRequest, err.Error())
+				return ctx, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, err.Error())
 			}
 
 			// only restrict a validator from unbonding it's self-delegation

diff --git a/x/axelarnet/types/types.go b/x/axelarnet/types/types.go
@@ -221,7 +221,7 @@ func ValidateIBCPath(path string) error {
 	// we only support direct IBC connections
 	pathSplit := strings.Split(path, "/")
 	if len(pathSplit) != 2 {
-		return fmt.Errorf(fmt.Sprintf("invalid IBC path %s", path))
+		return fmt.Errorf("invalid IBC path %s", path)
 	}
 
 	return nil

diff --git a/x/evm/keeper/grpc_query.go b/x/evm/keeper/grpc_query.go
@@ -462,7 +462,7 @@ func (q Querier) Bytecode(c context.Context, req *types.BytecodeRequest) (*types
 		return nil, status.Error(codes.NotFound, sdkerrors.Wrap(types.ErrEVM, fmt.Sprintf("could not retrieve bytecode for chain %s", req.Chain)).Error())
 	}
 
-	return &types.BytecodeResponse{Bytecode: fmt.Sprintf("0x" + common.Bytes2Hex(bytecode))}, nil
+	return &types.BytecodeResponse{Bytecode: "0x" + common.Bytes2Hex(bytecode)}, nil
 }
 
 // ERC20Tokens returns the ERC20 tokens registered for a chain

diff --git a/x/nexus/keeper/grpc_query.go b/x/nexus/keeper/grpc_query.go
@@ -265,7 +265,7 @@ func (q Querier) TransferRateLimit(c context.Context, req *types.TransferRateLim
 
 	chain, ok := q.keeper.GetChain(ctx, nexus.ChainName(req.Chain))
 	if !ok {
-		return nil, sdkerrors.Wrapf(sdkerrors.ErrNotFound, fmt.Errorf("chain %s not found", req.Chain).Error())
+		return nil, sdkerrors.Wrap(sdkerrors.ErrNotFound, fmt.Errorf("chain %s not found", req.Chain).Error())
 	}
 
 	rateLimit, found := q.keeper.getRateLimit(ctx, chain.Name, req.Asset)