-
Notifications
You must be signed in to change notification settings - Fork 16
hoodi deployment #120
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
base: master
Are you sure you want to change the base?
hoodi deployment #120
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ see 7 files with indirect coverage changes 🚀 New features to boost your workflow:
|
// https://hoodi.etherscan.io/address/0xcfc84cED34265A021C2652292dDe48c64A716e47 | ||
return 0xcfc84cED34265A021C2652292dDe48c64A716e47; | ||
} | ||
|
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.
Why isn't there a Timelock contract in this chain?
return 0xc7cc160b58F8Bb0baC94b80847E2CF2800565C50; | ||
} else if (block.chainid == hoodi) { | ||
// No Lido withdrawal queue on Hoodi testnet, using stETH address as fallback | ||
return 0xA1e64F52b83C9D0c2b2B5b7e16c8D5b9B3d8e4EA; |
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.
Is stETH contract interchangeable with LidoWithdrawalQueue?
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.
Also that address is a EOA. stETH address is 0x3508A952176b3c15387C97BE809eaffB1982176a
// https://holesky.etherscan.io/address/0xB028194785178a94Fe608994A4d5AD84c285A640 | ||
return 0xB028194785178a94Fe608994A4d5AD84c285A640; | ||
} else if (block.chainid == hoodi) { | ||
// Deployed ValidatorTicket address |
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.
Add explorer link for consistency:
https://hoodi.etherscan.io/address/0x6E0225679209459a355297A43B61cAFbC1Ca8d46
return address(0); // TODO | ||
} else if (block.chainid == hoodi) { | ||
// https://hoodi.etherscan.io/address/0x7aD345dC8583015f31E8f03CAE9cA09706514349 | ||
return 0x7aD345dC8583015f31E8f03CAE9cA09706514349; |
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.
Contract not verified
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.
As mentioned in other repo, why some contracts here are deployed without proxy? Also there are a few contracts that are not verified
No description provided.