more fixes

engine/engine.go

@@ -307,7 +307,11 @@ func (e *EngineAPI) NewPayloadV2(payload eth.ExecutionPayload) (*eth.PayloadStat
 }
 
 // NewPayloadV3 ensures the payload's block hash is present in the block store.
-func (e *EngineAPI) NewPayloadV3(payload eth.ExecutionPayload, _ []common.Hash, _ *common.Hash) (*eth.PayloadStatusV1, error) { //nolint:gocritic
+func (e *EngineAPI) NewPayloadV3(
+	payload eth.ExecutionPayload, //nolint:gocritic
+	_ []common.Hash,
+	_ *common.Hash,
+) (*eth.PayloadStatusV1, error) {
 	// TODO use the parentBeaconBlockRoot, etc.
 	e.lock.Lock()
 	defer e.lock.Unlock()

integrations/integrations.go

@@ -272,7 +272,7 @@ func startOPDevnet(
 		if !common.IsHexAddress(l1UserAddressStr) {
 			return errors.New("l1 user address is not a valid hex address")
 		}
-		balance, ok := new(big.Int).SetString("0x152D02C7E14AF6800000", 16) // 100,000 ETH
+		balance, ok := new(big.Int).SetString("0x152D02C7E14AF6800000", 0) // 100,000 ETH
 		if !ok {
 			return errors.New("failed to parse balance as big int")
 		}
@@ -491,7 +491,7 @@ func startMonomerNode(
 	trieDB := triedb.NewDatabase(rawDB, nil)
 	env.DeferErr("close trieDB", trieDB.Close)
 	snapshotTree, err := snapshot.New(snapshot.Config{
-		CacheSize: defaultCacheSize, // TODO
+		CacheSize: defaultCacheSize,
 	}, rawDB, trieDB, common.Hash{})
 	if err != nil {
 		return fmt.Errorf("new snapshot tree: %v", err)

opdevnet/l1.go

@@ -62,7 +62,7 @@ func (cfg *L1Config) Run(ctx context.Context, env *environment.Env, logger log.L
 	})
 	gethInstance, err := geth.InitL1(
 		cfg.BlockTime,
-		8,
+		8, //nolint:mnd
 		cfg.Genesis,
 		clock.NewAdvancingClock(time.Second), // Arbitrary working duration. Eventually consumed by geth lifecycle instances.,
 		cfg.BlobsDirPath,

testutils/utils.go

@@ -90,7 +90,7 @@ func GenerateEthTxs(t *testing.T) (*gethtypes.Transaction, *gethtypes.Transactio
 }
 
 func GenerateEthBridgeDepositTx(t *testing.T, userAddr common.Address, amount *big.Int) *gethtypes.Transaction {
-	// We should techincally use the ABI for the L2StandardBridge, but we only have the L1 bindings and they work fine here.
+	// We should technically use the ABI for the L2StandardBridge, but we only have the L1 bindings and they work fine here.
 	standardBridgeABI, err := abi.JSON(strings.NewReader(opbindings.L1StandardBridgeMetaData.ABI))
 	require.NoError(t, err)
 	rng := rand.New(rand.NewSource(1234))
@@ -108,7 +108,7 @@ func GenerateEthBridgeDepositTx(t *testing.T, userAddr common.Address, amount *b
 }
 
 func GenerateERC20DepositTx(t *testing.T, tokenAddr, userAddr common.Address, amount *big.Int) *gethtypes.Transaction {
-	// We should techincally use the ABI for the L2StandardBridge, but we only have the L1 bindings and they work fine here.
+	// We should technically use the ABI for the L2StandardBridge, but we only have the L1 bindings and they work fine here.
 	standardBridgeABI, err := abi.JSON(strings.NewReader(opbindings.L1StandardBridgeMetaData.ABI))
 	require.NoError(t, err)
 	rng := rand.New(rand.NewSource(1234))

x/rollup/keeper/deposits.go

@@ -103,7 +103,7 @@ func (k *Keeper) processCrossDomainMessage(ctx sdk.Context, txData []byte) (*sdk
 	}
 	// It is a bit lazy to use the L1StandardBridge ABI instead of the L2StandardBridge ABI, but the bindings
 	// package doesn't have the latter. Plus, we know that the functions we want are implemented in the StandardBridge
-	// abstract contract and aren't overriden.
+	// abstract contract and aren't overridden.
 	standardBridgeABI, err := abi.JSON(strings.NewReader(crossdomainbindings.L1StandardBridgeMetaData.ABI))
 	if err != nil {
 		return nil, fmt.Errorf("failed to parse StandardBridge ABI: %v", err)