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.

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

βž• Add Gravity Alpha Main Network Deployment #129

Merged
merged 5 commits into from
Aug 18, 2024

Conversation

oyyblin
Copy link
Contributor

@oyyblin oyyblin commented Aug 16, 2024

πŸ•“ Changelog

Add Gravity Alpha main network deployment:

Verification

Compare the keccak256 hash of the runtime bytecode with the canonical keccak256 hash of the runtime bytecode here (0xbd8a7ea8cfca7b4e5f5041d7d4b17bc317c5ce42cfbc42066a00cf26b43eb53f):

~$ cast keccak $(cast code 0xba5Ed099633D3B313e4D5F7bdc1305d3c28ba5Ed --rpc-url https://rpc.gravity.xyz)
0xbd8a7ea8cfca7b4e5f5041d7d4b17bc317c5ce42cfbc42066a00cf26b43eb53f

🐢 Cute Animal Picture

image

@pcaversaccio pcaversaccio changed the title βž• Add Gravity Alpha Mainnet and Sepolia testnet deployments βž• Add Gravity Alpha Test and Main Network Deployments Aug 17, 2024
@pcaversaccio pcaversaccio self-requested a review August 17, 2024 09:43
@pcaversaccio pcaversaccio self-assigned this Aug 17, 2024
@pcaversaccio pcaversaccio added documentation πŸ“– Improvements or additions to documentation dependencies πŸ” Pull requests that update a dependency file new deployment βž• New deployment request labels Aug 17, 2024
@pcaversaccio pcaversaccio added this to the 1.0.0 milestone Aug 17, 2024
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio changed the title βž• Add Gravity Alpha Test and Main Network Deployments βž• Add Gravity Alpha Main Network Deployment Aug 17, 2024
Copy link
Owner

@pcaversaccio pcaversaccio left a comment

Choose a reason for hiding this comment

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

Thx for the PR. I quickly amended some formatting for consistency reasons. There is a bug in the latest prettier-plugin-solidity 1.4.0 version (see here), which I bump as part of this PR as well. Waiting for the fix before merging.

I don't know who (maybe you?) but someone fucked up the presigned deployment transaction on your Sepolia test network as it seems the gas calculations are not EVM equivalent: https://explorer-sepolia.gravity.xyz/tx/0xb6274b80bc7cda162df89894c7748a5cb7ba2eaa6004183c41a1837c3b072f1e. I can't add an even bigger warning...

image
image

This means that CreateX will never exist on your Sepolia test network unfortunately, and thus I removed it from the PR.

Signed-off-by: Pascal Marco Caversaccio <[email protected]>
@pcaversaccio pcaversaccio merged commit befb986 into pcaversaccio:main Aug 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies πŸ” Pull requests that update a dependency file documentation πŸ“– Improvements or additions to documentation new deployment βž• New deployment request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants