Commit f5539149 authored by Mark Tyneway's avatar Mark Tyneway

op-chain-ops: make protocol versions warn instead of err

Instead of erroring it should warn. Otherwise we need to
use a pointer and use `nil` to differentiate from the 0 value.
parent 1e26ab9c
...@@ -353,10 +353,10 @@ func (d *DeployConfig) Check() error { ...@@ -353,10 +353,10 @@ func (d *DeployConfig) Check() error {
return fmt.Errorf("L2 block time (%d) is larger than L1 block time (%d)", d.L2BlockTime, d.L1BlockTime) return fmt.Errorf("L2 block time (%d) is larger than L1 block time (%d)", d.L2BlockTime, d.L1BlockTime)
} }
if d.RequiredProtocolVersion == (params.ProtocolVersion{}) { if d.RequiredProtocolVersion == (params.ProtocolVersion{}) {
return fmt.Errorf("%w: RequiredProtocolVersion cannot be empty", ErrInvalidDeployConfig) log.Warn("RequiredProtocolVersion is empty")
} }
if d.RecommendedProtocolVersion == (params.ProtocolVersion{}) { if d.RecommendedProtocolVersion == (params.ProtocolVersion{}) {
return fmt.Errorf("%w: RecommendedProtocolVersion cannot be empty", ErrInvalidDeployConfig) log.Warn("RecommendedProtocolVersion is empty")
} }
return nil return nil
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment