Skip to content

Conversation

tdenisenko
Copy link

No description provided.

Copy link

codecov bot commented Jul 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

// https://hoodi.etherscan.io/address/0xcfc84cED34265A021C2652292dDe48c64A716e47
return 0xcfc84cED34265A021C2652292dDe48c64A716e47;
}

Copy link
Contributor

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;
Copy link
Contributor

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?

Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

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

return address(0); // TODO
} else if (block.chainid == hoodi) {
// https://hoodi.etherscan.io/address/0x7aD345dC8583015f31E8f03CAE9cA09706514349
return 0x7aD345dC8583015f31E8f03CAE9cA09706514349;
Copy link
Contributor

Choose a reason for hiding this comment

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

Contract not verified

Copy link
Contributor

@eladiosch eladiosch left a 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

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.

2 participants