blaine
Closing, implemented in this PR: https://github.com/ethereum-optimism/superchain-ops/pull/819
We should use `forge coverage` to get a lcov file. We can open that in an lcov reader. `lcov` has a cli.
Working on this in: https://github.com/ethereum-optimism/superchain-ops/pull/640
Keeping this issue open. I think we can use this issue to more broadly check if the api, state variable etc in this contract can be simplified for readability and...
Closing this issue as we're undertaking a larger refactor project that tries to support an arbitrary amount of nesting.
https://github.com/ethereum-optimism/superchain-registry/pull/897#discussion_r1957150033
OPCM does exist in scr now: https://github.com/ethereum-optimism/superchain-registry/blob/main/validation/standard/standard-versions-mainnet.toml#L22
Addressed with this PR: https://github.com/ethereum-optimism/superchain-ops/pull/803
Instead of adding a new variable to the `ChainInfo` struct for `.l2Chains`, we should introduce a new TOML property called `.l2ChainsLocal`. The `.l2Chains` and `.l2ChainsLocal` properties must **not** coexist in...
Closing this in favor of a different PR. In rebase hell with this one.