Skip to content

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Apr 23, 2021

Description
Removes the OVMSigner from the codebase, required for RLP regenesis

Additional context
Port of ethereum-optimism/go-ethereum#278

github-actions bot and others added 13 commits April 22, 2021 14:25
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
#555)

* stub out test

* fix: getXDomainCalldata argument order to match usage

* feat: completed test

* fix: working test
* export metrics in core-utils

* initialize metrics in batch submitters

* add changeset

* fix unit tests
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* feat[smock]: Add support for hardhat 2.2.0

* refactor[smock]: Clean up some repeated code

* fix[smock]: Dynamically resolve TransactionExecutionError

* fix[smock]: Fix linting error

* fix[smock]: Use older ethers version

* fix[smock]: Minor hack for backwards compat

* chore[smock]: Add changeset
* contracts: remove all trailing whitespace

* chore: add changeset
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Also reduce the diff

Co-authored-by: smartcontracts
@changeset-bot
Copy link

changeset-bot bot commented Apr 23, 2021

⚠️ No Changeset found

Latest commit: f0e03f2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@tynes tynes changed the title Port/ovmsigner l2geth: remove ovmsigner Apr 23, 2021
@tynes tynes closed this Apr 23, 2021
@tynes tynes deleted the port/ovmsigner branch April 23, 2021 18:33
agusduha pushed a commit that referenced this pull request Sep 25, 2025
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2025
* feat: fee splitter system (#469)


----

Co-authored-by: Disco <[email protected]>
Co-authored-by: Flux <[email protected]>
Co-authored-by: Chiin <[email protected]>

* fix: failing tests (#553)

* refactor: revenue sharing config (#538)

* fix: pre pr and semgrep (#565)

* fix: warnings (#571)

* fix: tests failing on fork environment (#575)

* fix: use encode call on constructors (#578)

* feat: add integration splitter test (#581)

* fix: integration test nits (#583)

* test: more coverage on splitter tests (#594)


---------

Co-authored-by: Chiin <[email protected]>

* fix: add missing operator fee vault field (#616)

* feat: invariants and tests (#595)

* fix: ir informationals (#609)

* fix: pre pr (#622)

* refactor: use cdm (#624)

* fix: update withdrawal gas limit value on check (#627)

* fix: remove rev share field from deploy op chain input struct (#628)

* refactor: remove initializer and vaults getter changes (#631)

* fix: remove immutable check over vaults (#634)


Co-authored-by: 0xng <[email protected]>
Co-authored-by: Funkornaut <[email protected]>
Co-authored-by: Joxess <[email protected]>

* fix: audit findings (#658)

* fix: pre pr

* feat: add user guide docs for rev sharing on op deployer (#666)

* docs: add create2 comment (#667)

---------

Co-authored-by: Funkornaut <[email protected]>
Co-authored-by: Flux <[email protected]>
Co-authored-by: Chiin <[email protected]>
Co-authored-by: Simon Something /DrGoNoGo <[email protected]>
Co-authored-by: 0xng <[email protected]>
Co-authored-by: Joxess <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants