Skip to content

Conversation

@aapclark
Copy link
Contributor

update readme and verification instructions

add template envs for testnet and mainnet deployments

@aapclark aapclark force-pushed the improve-evm-deployment-scripts branch from 235cdb3 to 10db9ee Compare October 14, 2025 20:47
@aapclark aapclark marked this pull request as ready for review October 14, 2025 20:47
update readme and verification instructions

add template envs for testnet and mainnet deployments

update cspell word list

update verification document
@aapclark aapclark force-pushed the improve-evm-deployment-scripts branch from 839b93a to 2164cf1 Compare October 17, 2025 17:13
Comment on lines 34 to 35
BRIDGE_INIT_GOV_CHAIN_ID=0x1
BRIDGE_INIT_GOV_CONTRACT=0x0000000000000000000000000000000000000000000000000000000000000004
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could reuse the definitions?

Suggested change
BRIDGE_INIT_GOV_CHAIN_ID=0x1
BRIDGE_INIT_GOV_CONTRACT=0x0000000000000000000000000000000000000000000000000000000000000004
BRIDGE_INIT_GOV_CHAIN_ID=$INIT_GOV_CHAIN_ID
BRIDGE_INIT_GOV_CONTRACT=$INIT_GOV_CONTRACT

And similar things could be done for other parameters like the Wormhole chain id too.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea. I'll make those changes.

djb15
djb15 previously approved these changes Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants