Skip to content

Replace Geth fork with original Geth#886

Closed
m-Peter wants to merge 1 commit into
onflow:masterfrom
m-Peter:mpeter/replace-geth-fork
Closed

Replace Geth fork with original Geth#886
m-Peter wants to merge 1 commit into
onflow:masterfrom
m-Peter:mpeter/replace-geth-fork

Conversation

@m-Peter
Copy link
Copy Markdown
Contributor

@m-Peter m-Peter commented Aug 11, 2025

Work Towards: onflow/flow-go#7232

Description

We want to move away from our fork of Geth (https://github.com/onflow/go-ethereum) and use the official Geth repository.

Related PRs for replacing Geth fork:


For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the standards mentioned here.
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Contributor

@peterargue peterargue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, but let's wait to merge until the flow-go PR is merged and tagged and the updated version is used here.

@turbolent
Copy link
Copy Markdown
Member

Looks like I missed this open PR and duplicated the work in #893. I guess this PR is no longer needed now

@m-Peter
Copy link
Copy Markdown
Contributor Author

m-Peter commented Aug 20, 2025

Closing in favor of #893

@m-Peter m-Peter closed this Aug 20, 2025
@m-Peter m-Peter deleted the mpeter/replace-geth-fork branch October 20, 2025 08:34
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.

5 participants