feat: Add Conditional L2 Deployer#18864
Merged
maurelian merged 15 commits intoethereum-optimism:developfrom Feb 13, 2026
Merged
feat: Add Conditional L2 Deployer#18864maurelian merged 15 commits intoethereum-optimism:developfrom
maurelian merged 15 commits intoethereum-optimism:developfrom
Conversation
…eployer-sync chore: l2cm conditional deployer sync
Contributor
|
This pr has been automatically marked as stale and will be closed in 5 days if no updates |
…lag (#836) * refactor: replace create2 deployer for arachnid * feat: add ConditionalDeployer as predeploy and add L2CM dev flag * fix: add pre-pr fix * fix: remove unnecessary auth in conditional deployer * feat: add custom error for deployment revert * test: add l2cm e2e apply test case
Contributor
|
/ci authorize |
maurelian
reviewed
Feb 6, 2026
packages/contracts-bedrock/interfaces/L2/IConditionalDeployer.sol
Outdated
Show resolved
Hide resolved
* fix: remove fork test * feat: add common test to cond deployer test * fix: follow constant style guide
Contributor
|
/ci authorize |
0xOneTony
reviewed
Feb 9, 2026
0xiamflux
commented
Feb 9, 2026
0xiamflux
commented
Feb 9, 2026
0xiamflux
commented
Feb 9, 2026
* fix: conditional deployer comment in apply test * fix: arachnid reference comment * fix: add msg.value to conditional deployer * fix: deterministic deployment proxy comment
sync: l2cm conditional deployer
Contributor
|
/ci authorize 290addb |
maurelian
reviewed
Feb 9, 2026
* fix: add code length check after deploy * fix: pre pr fix
Contributor
|
/ci authorize |
maurelian
reviewed
Feb 10, 2026
maurelian
reviewed
Feb 10, 2026
maurelian
reviewed
Feb 10, 2026
maurelian
reviewed
Feb 10, 2026
0xniha
reviewed
Feb 10, 2026
0xniha
reviewed
Feb 11, 2026
* fix: make deploy non payable * test: add missing getters test * fix: add missing natspec * feat: add todo issue for devfeatures cyclic import * chore: pre pr run
Contributor
Plan expiredYour subscription has expired. Please renew your subscription to continue using CI/CD integration and other features. |
Contributor
|
/ci authorize |
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
maurelian
reviewed
Feb 11, 2026
Contributor
|
/ci authorize |
0xiamflux
commented
Feb 11, 2026
Contributor
|
/ci authorize |
maurelian
approved these changes
Feb 11, 2026
0xOneTony
reviewed
Feb 11, 2026
* fix: remove underscore from local variable * fix: remove address payable from deterministicDeploymentProxy * refactor: rename dummy storage variable in test and add internal getExpectedImplementation helper
Contributor
|
/ci authorize |
mds1
approved these changes
Feb 12, 2026
github-merge-queue
bot
removed this pull request from the merge queue due to no response for status checks
github-merge-queue
bot
removed this pull request from the merge queue due to no response for status checks
Merged
via the queue into
ethereum-optimism:develop
with commit Feb 13, 2026
75071b9
209 of 210 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Conditional Deployer
Adds a wrapper around Create2Deployer to be used during Network Upgrade Transactions, allowing for non-reverting deterministic deployments even when address collisions occur.
Changes
ConditionalDeployer.sol
Create2Deployerfor conditional deployment of contracts, in case of an address collision simply returns the address without reverting.Reopening #18295
Reference Issue: #18828