Skip to content

Commit

Permalink
fix(op-deployer): don't deploy dachallenge contract when using Generi…
Browse files Browse the repository at this point in the history
…cCommitment (#14248)
  • Loading branch information
samlaf authored and maurelian committed Feb 11, 2025
1 parent 92adbc7 commit 0845e0d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion op-deployer/pkg/deployer/pipeline/alt_da.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"fmt"
"math/big"

altda "github.com/ethereum-optimism/optimism/op-alt-da"
"github.com/ethereum-optimism/optimism/op-deployer/pkg/deployer/opcm"
"github.com/ethereum-optimism/optimism/op-deployer/pkg/deployer/state"
"github.com/ethereum/go-ethereum/common"
Expand Down Expand Up @@ -48,7 +49,7 @@ func DeployAltDA(env *Env, intent *state.Intent, st *state.State, chainID common
}

func shouldDeployAltDA(chainIntent *state.ChainIntent, chainState *state.ChainState) bool {
if !chainIntent.DangerousAltDAConfig.UseAltDA {
if !(chainIntent.DangerousAltDAConfig.UseAltDA && chainIntent.DangerousAltDAConfig.DACommitmentType == altda.KeccakCommitmentString) {
return false
}

Expand Down

0 comments on commit 0845e0d

Please sign in to comment.