Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

contracts-bedrock: fix build #3114

Merged
merged 2 commits into from
Jul 28, 2022
Merged

contracts-bedrock: fix build #3114

merged 2 commits into from
Jul 28, 2022

Commits on Jul 28, 2022

  1. contracts-bedrock: fix build 

    Standardize on the upgradable initializable since the contracts
are technically upgradable it is more clear. There are no real
implementation differences between the upgradable and standard
initializable implementations. `OwnableUpgradable` is `initializable`
imported from the upgradable package, and the `L2OutputOracle`
inherits from `OwnableUpgradable`. This means that the only
way to standardize on a single implementation of `Initializable`
is to use the upgradable version.

This also bumps the version of the openzeppelin contracts
dependency because they refactored the initializable implementation
and made it easier to understand.

https://github.com/OpenZeppelin/openzeppelin-contracts-upgradeable/releases/tag/v4.7.0
OpenZeppelin/openzeppelin-contracts#3450
    @tynes
    tynes committed Jul 28, 2022
    Configuration menu
    Copy the full SHA
    a162b7f View commit details
    Browse the repository at this point in the history

  2. op-bindings: regenerate

    @tynes
    tynes committed Jul 28, 2022
    Configuration menu
    Copy the full SHA
    24a1e12 View commit details
    Browse the repository at this point in the history