refactor: extract op-reth binary to separate crate

[klkvr]

ref #9439

I've changed OpChainSpecParser to return an inner ChainSpec for now to avoid introducing more complex bounds on cli types. We can change it back once we figure out how the bounds should look like on underlying types

[emhane]

great job

[emhane]

do mind trying to fix the commit history on this branch? usually does it by checking out a backup branch of your diff from main, then on this branch git reset --hard main, then cherry pick from backup branch

[klkvr]

do mind trying to fix the commit history on this branch? usually does it by checking out a backup branch of your diff from main, then on this branch git reset --hard main, then cherry pick from backup branch

you mean squashing the diff into less number of commits without merge commits?

[joshieDo]

no need to touch up the history of the branch imo, this all gets squashed into main anyway

[emhane]

as long as the diff in 'Files changed' isn't showing unrelated changes, and other people are comfortable reviewing (@joshieDo), then I'm fine with unrelated changes showing in GitHub commit history for the pr

[emhane]

think you will have to add a new job for op-reth, we still want to check lint for op code

[klkvr]

added a step to run cargo check on it with optimism feature

[mattsse]

hell yeah, let's try this again hehe

completed