Skip to content

feat: Optimism Goerli Chainspec + Engine API mods#4

Closed
clabby wants to merge 192 commits intoop-rs:refcell/compilingfrom
clabby:clabby/goerli-genesis
Closed

feat: Optimism Goerli Chainspec + Engine API mods#4
clabby wants to merge 192 commits intoop-rs:refcell/compilingfrom
clabby:clabby/goerli-genesis

Conversation

@clabby
Copy link

@clabby clabby commented Mar 5, 2023

Overview

Note
WIP

Adds in the chainspec for Optimism Goerli as well as finalizes the Engine API modifications defined in Optimism's Execution Engine Specs / the op-geth diff.

Also merges in the changes made upstream since the start of the project.

tomarST and others added 30 commits February 26, 2023 00:40
Co-authored-by: Bjerg <onbjerg@users.noreply.github.com>
Co-authored-by: Roman Krasiuk <rokrassyuk@gmail.com>
Co-authored-by: Oliver Nordbjerg <hi@notbjerg.me>
Co-authored-by: lambdaclass-user <github@lambdaclass.com>
Co-authored-by: Dan Cline <6798349+Rjected@users.noreply.github.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
…adigmxyz#1474)

Co-authored-by: Georgios Konstantopoulos <me@gakonst.com>
Co-authored-by: Roman Krasiuk <rokrassyuk@gmail.com>
@clabby clabby force-pushed the clabby/goerli-genesis branch 4 times, most recently from cedb358 to 5eb1b2a Compare March 19, 2023 22:01
@clabby clabby force-pushed the clabby/goerli-genesis branch from 5eb1b2a to 7d8d2b3 Compare March 19, 2023 22:56
@clabby clabby closed this Mar 19, 2023
clabby added a commit that referenced this pull request Aug 12, 2023
clabby added a commit that referenced this pull request Aug 13, 2023
Resolution checkpoint

Resolution checkpoint #2

Resolution checkpoint #3

x

Resolution checkpoint #4

Resolution checkpoint #5

Resolution checkpoint #6

Resolution checkpoint #7

Resolution checkpoint #8

Resolve checkpoint #9 (transaction primitive)

Resolve checkpoint #10 (rpc api transactions)

Resolve checkpoint #11 (building w/o feature flag)

Start review

Compiling with and without `optimism` feature flag

Remove `DepositTx` from txpool mock tests, they never go into the txpool

fmt

code lint

fix signature tests

Co-authored-by: nicolas <48695862+merklefruit@users.noreply.github.com>

Use free CI runners (revert before upstream)

Co-authored-by: refcell <abigger87@gmail.com>

Signature test fixes

Co-authored-by refcell <abigger87@gmail.com>

Fix Receipt proptest

Co-authored-by BB <brian.t.bland@gmail.com>

lint

Fix variable-length compact for txtype/transaction

Co-authored-by: Brian Bland <brian.t.bland@gmail.com>

Fix basefee tests

Remove unnecessary rpc deps

Co-authored-by: Brian Bland <brian.t.bland@gmail.com>
Co-authored-by: refcell <abigger87@gmail.com>
Co-authored-by: nicolas <48695862+merklefruit@users.noreply.github.com>
Co-authored-by: Roberto <bayardo@alum.mit.edu>
clabby added a commit that referenced this pull request Aug 13, 2023
Resolution checkpoint

Resolution checkpoint #2

Resolution checkpoint #3

x

Resolution checkpoint #4

Resolution checkpoint #5

Resolution checkpoint #6

Resolution checkpoint #7

Resolution checkpoint #8

Resolve checkpoint #9 (transaction primitive)

Resolve checkpoint #10 (rpc api transactions)

Resolve checkpoint #11 (building w/o feature flag)

Start review

Compiling with and without `optimism` feature flag

Remove `DepositTx` from txpool mock tests, they never go into the txpool

fmt

code lint

fix signature tests

Co-authored-by: nicolas <48695862+merklefruit@users.noreply.github.com>

Use free CI runners (revert before upstream)

Co-authored-by: refcell <abigger87@gmail.com>

Signature test fixes

Co-authored-by refcell <abigger87@gmail.com>

Fix Receipt proptest

Co-authored-by BB <brian.t.bland@gmail.com>

lint

Fix variable-length compact for txtype/transaction

Co-authored-by: Brian Bland <brian.t.bland@gmail.com>

Fix basefee tests

Remove unnecessary rpc deps

Co-authored-by: Brian Bland <brian.t.bland@gmail.com>
Co-authored-by: refcell <abigger87@gmail.com>
Co-authored-by: nicolas <48695862+merklefruit@users.noreply.github.com>
Co-authored-by: Roberto <bayardo@alum.mit.edu>
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.