alloy-rs / op-alloy

Transports, Middleware, and Networks for Optimism built with Alloy
https://alloy-rs.github.io/op-alloy/
Apache License 2.0
38 stars 30 forks source link

Rename `TxDeposit::signature` to `pre_bedrock_signature` #203

Closed emhane closed 1 week ago

emhane commented 2 weeks ago

Motivation

Fixes semantics of TxDeposit to differentiate between ovm and evm chain

Solution

Renames TxDeposit::signature to pre_bedrock_signature

PR Checklist

mattsse commented 1 week ago

yeah this is actually included by all deposit transactions after bedrock, so we can keep it as is