What does this pull request do? Explain your changes. (required)
Adds arbitrumMainnet config values for deployment. The values of all params are the same as those used on mainnet except for:
The owner of the Governor is the L2 governance multisig specified in LIP-73
The inflation param (i.e. starting inflation rate) in the Minter is calculated based on a projection of what it will be on L1 in the round at which the LIP-73 upgrade is executed. See the comments for the arbitrumMainnet config in migrations.config.js for an explanation for how it was calculated
A few other minor updates are included in this PR as well which are explained by each of the commits.
Specific updates (required)
See commit history.
How did you test each of these updates (required)
I ran yarn deploy with a local HH network. Note this does not test the network conditional logic that would only be triggered when connected to arbitrumMainnet.
What does this pull request do? Explain your changes. (required)
Adds arbitrumMainnet config values for deployment. The values of all params are the same as those used on mainnet except for:
inflation
param (i.e. starting inflation rate) in the Minter is calculated based on a projection of what it will be on L1 in the round at which the LIP-73 upgrade is executed. See the comments for the arbitrumMainnet config inmigrations.config.js
for an explanation for how it was calculatedA few other minor updates are included in this PR as well which are explained by each of the commits.
Specific updates (required)
See commit history.
How did you test each of these updates (required)
I ran
yarn deploy
with a local HH network. Note this does not test the network conditional logic that would only be triggered when connected to arbitrumMainnet.Does this pull request close any open issues?
N/A
Checklist:
yarn test
pass