optimism
optimism copied to clipboard
Parse Solidity Deploy Config Lazily
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.
Hi @tynes, Is it still accurate? Happy to jump on this one :D
Is this still something we want to do? @tynes