Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Deploy Snaxchain mainnet #4078

Merged
merged 3 commits into from
Aug 16, 2024
Merged

Deploy Snaxchain mainnet #4078

merged 3 commits into from
Aug 16, 2024

Conversation

bruce-riley
Copy link
Contributor

This PR enables Snaxchain mainnet support. It also updates the testnet deployment since they changed the EVM chain ID for it.

The mainnet contracts are here:

The testnet contracts are here:

@bruce-riley bruce-riley marked this pull request as ready for review August 15, 2024 15:45
panoel
panoel previously approved these changes Aug 15, 2024
View reviewed changes
evan-gray
evan-gray previously approved these changes Aug 16, 2024
View reviewed changes
ethereum/env/.env.snaxchain.testnet Outdated Show resolved Hide resolved
node/pkg/governor/mainnet_tokens_test.go Outdated Show resolved Hide resolved
@bruce-riley bruce-riley dismissed stale reviews from evan-gray and panoel via e564f10 August 16, 2024 14:13
evan-gray
evan-gray previously approved these changes Aug 16, 2024
View reviewed changes
johnsaigle
johnsaigle reviewed Aug 16, 2024
View reviewed changes
Copy link
Contributor

@johnsaigle johnsaigle left a comment

Choose a reason for hiding this comment

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

LGTM pending Solano's feedback on Dirk's note.

@evan-gray evan-gray requested review from johnsaigle and djb15 August 16, 2024 17:39
@evan-gray evan-gray merged commit 22eecc6 into wormhole-foundation:main Aug 16, 2024
24 checks passed
@evan-gray evan-gray deleted the deploy_snaxchain branch August 16, 2024 17:44
Mikhai56
Mikhai56 reviewed Aug 16, 2024
View reviewed changes
Copy link

@Mikhai56 Mikhai56 left a comment

Choose a reason for hiding this comment

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

gobro

@aadam-10 aadam-10 mentioned this pull request Aug 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mikhai56 Mikhai56 Mikhai56 approved these changes

@evan-gray evan-gray evan-gray approved these changes

@djb15 djb15 djb15 approved these changes

@panoel panoel panoel left review comments

@kev1n-peters kev1n-peters Awaiting requested review from kev1n-peters kev1n-peters is a code owner

@SEJeff SEJeff Awaiting requested review from SEJeff SEJeff is a code owner

@kcsongor kcsongor Awaiting requested review from kcsongor kcsongor is a code owner

@a5-pickle a5-pickle Awaiting requested review from a5-pickle a5-pickle is a code owner

@gator-boi gator-boi Awaiting requested review from gator-boi gator-boi is a code owner

@claudijd claudijd Awaiting requested review from claudijd claudijd is a code owner

@johnsaigle johnsaigle Awaiting requested review from johnsaigle johnsaigle is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@bruce-riley @johnsaigle @djb15 @panoel @evan-gray @Mikhai56