optimism icon indicating copy to clipboard operation
optimism copied to clipboard

Parse Solidity Deploy Config Lazily

Open tynes opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe.

The DeployConfig contract is used to parse the JSON deploy config files. The problem is that some deploy config values are for testnet only features, like the fault proof. Since all config is parsed in the constructor, this means that all deploy config files need to contain all of the config values even if the config values aren't used in a particular script execution. The deploy config contract should be updated to parse lazily, meaning that it no longer reads the entire JSON file in the constructor but instead it has a getter for each value and reads the JSON file inside of each getter for the particular value.

tynes avatar Oct 17 '23 03:10 tynes

Hi @tynes, Is it still accurate? Happy to jump on this one :D

Ekzer avatar Dec 13 '23 23:12 Ekzer

Is this still something we want to do? @tynes

smartcontracts avatar Feb 05 '24 01:02 smartcontracts