address review comments

e2e/e2e.go

@@ -76,6 +76,7 @@ func Run(
 		"--monomer.sequencer",
 	))
 	// Use a headless dlv instance to debug the testapp.
+	// Connect using `dlv connect :2345`.
 	//nolint:gocritic // commentedOutCode
 	/*
 		appCmd := setupCmd(exec.CommandContext(ctx,

e2e/stack_test.go

@@ -245,7 +245,6 @@ func ethRollupFlow(t *testing.T, stack *e2e.StackConfig) {
 	// create a withdrawal tx to withdraw the deposited amount from L2 back to L1
 	withdrawalTx := e2e.NewWithdrawalTx(0, common.Address(userCosmosETHAddress), userETHAddress, withdrawalAmount, new(big.Int).SetUint64(params.TxGas))
 
-	t.Logf("FROM TEST: %s", userCosmosAddr)
 	baseAccount := queryAccount(t, stack, userCosmosAddr)
 	l2ChainID, err := stack.MonomerClient.ChainID(stack.Ctx)
 	require.NoError(t, err)

engine/engine.go

@@ -58,7 +58,6 @@ func (e *EngineAPI) ForkchoiceUpdatedV1(
 	fcs eth.ForkchoiceState, //nolint:gocritic
 	pa *eth.PayloadAttributes,
 ) (*eth.ForkchoiceUpdatedResult, error) {
-	// TODO should this be called after Ecotone?
 	return e.ForkchoiceUpdatedV3(ctx, fcs, pa)
 }
 

opdevnet/opdevnet_test.go

@@ -124,8 +124,6 @@ func TestOPDevnet(t *testing.T) {
 
 	l2SlotDuration := time.Duration(opConfig.Node.Rollup.BlockTime) * time.Second
 
-	t.Log("L1 and OP are setup")
-
 	// Confirm safe blocks are incrementing to ensure the batcher is posting blocks to the DA layer.
 	for l2Instance.Backend.BlockChain().CurrentSafeBlock().Number.Uint64() < 3 {
 		time.Sleep(l2SlotDuration)