Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update optimism monorepo dependency to 874c0e9 #81

Merged
merged 19 commits into from
Oct 1, 2024

Conversation

mininny
Copy link
Collaborator

@mininny mininny commented Sep 24, 2024

Description

Update Optimism monorepo's dependency to ethereum-optimism/optimism@874c0e9.
There are too many diff between the last dependency version(ethereum-optimism/optimism@457f33f...874c0e9), but many of the changes are from the fault proof audit fixes(see ethereum-optimism/optimism#11503)

Includes the following changes

  • Update gitmodule of optimism and forge-std
  • Update code changes from the monorepo dependency
  • Mirror new/updated e2e tests from op-e2e
  • Update Deploy scripts, mostly mirrored from monorepo updates
  • Update CircleCI pipeline, mostly mirrored from monorepo updates
    • Remove pnpm usage
    • Update optimism ci-builder docker image
  • Update geth abigen dependency and generated bindings
  • Update op-program-test data and test data generation script

Tests

All existing github workflow tests and circleci tests are passing.
Newly added op-e2e tests are passing.

Additional context

@mininny mininny force-pushed the feature/mininny/optimism-308ce74c7 branch from 012d01e to 7aef902 Compare September 24, 2024 21:32
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2024

Codecov Report

Attention: Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.

Project coverage is 72.01%. Comparing base (636210a) to head (0a894ae).

Files with missing lines Patch % Lines
rvgo/bindings/preimageoracle.go 0.00% 17 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   72.30%   72.01%   -0.30%     
==========================================
  Files          16       16              
  Lines        2723     2734      +11     
==========================================
  Hits         1969     1969              
- Misses        670      681      +11     
  Partials       84       84              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mininny mininny force-pushed the feature/mininny/optimism-308ce74c7 branch 5 times, most recently from 5e220bd to dd515ed Compare September 25, 2024 16:23
@mininny mininny force-pushed the feature/mininny/optimism-308ce74c7 branch from dd515ed to 178e37a Compare September 25, 2024 16:30
@mininny mininny force-pushed the feature/mininny/optimism-308ce74c7 branch from 989ba18 to 31f16fa Compare September 25, 2024 22:36
@mininny mininny force-pushed the feature/mininny/optimism-308ce74c7 branch from 1457eef to 7279f1e Compare September 26, 2024 03:04
@mininny mininny changed the title [WIP] Update optimism monorepo dependency to 308ce74c7 [WIP] Update optimism monorepo dependency to 874c0e9 Sep 26, 2024
@mininny mininny changed the title [WIP] Update optimism monorepo dependency to 874c0e9 Update optimism monorepo dependency to 874c0e9 Sep 26, 2024
@mininny mininny marked this pull request as ready for review September 26, 2024 03:13
@mininny mininny added this pull request to the merge queue Oct 1, 2024
Merged via the queue into master with commit 25feabf Oct 1, 2024
8 checks passed
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.

4 participants