-
Notifications
You must be signed in to change notification settings - Fork 195
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.
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
feat(contract_manager): script for EVM feeds contract deployment #1254
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
06fdc31
to
aafb26b
Compare
aafb26b
to
f27ff01
Compare
@@ -404,7 +404,7 @@ export class EvmEntropyContract extends Storable { | |||
return { | |||
chain: this.chain.getId(), | |||
address: this.address, | |||
type: EvmPriceFeedContract.type, | |||
type: EvmEntropyContract.type, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was a bug in json serialization of entropy contracts
@@ -44,18 +44,18 @@ const parser = yargs(hideBin(process.argv)) | |||
}, | |||
}); | |||
|
|||
async function run_if_not_cached( | |||
cache_key: string, | |||
async function runIfNotCached( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
format the naming style
|
||
if (config.type === "stable") { | ||
console.log(`Syncing mainnet guardian sets for ${chain.getId()}...`); | ||
// TODO: Add a way to pass gas configs to this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is a generic method I can't easily add the gas options to it. Something to refactor later in the future.
This change adds a script to replace truffle deployment scripts. This gives us more control and flexibility over the deployment process.
f27ff01
to
5f8ebfb
Compare
@@ -598,7 +598,7 @@ export class EvmPriceFeedContract extends PriceFeedContract { | |||
pythContract.methods.updatePriceFeeds(priceFeedUpdateData); | |||
const gasEstiamte = await transactionObject.estimateGas({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gasEstimate
artifactName: string, | ||
deployArgs: any[] // eslint-disable-line @typescript-eslint/no-explicit-any | ||
): Promise<string> { | ||
const cache = existsSync(CACHE_FILE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use runIfNotCached here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's a good idea. Let me do it in a separate PR (because this one is already very big)
This change adds a script to replace truffle deployment scripts. This gives us more control and flexibility over the deployment process.