-
Notifications
You must be signed in to change notification settings - Fork 438
feat: merge Moocow and ELIP5 into main #1425
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
Conversation
* docs: add docs for moocow methods and move EigenPod to inheritdoc syntax * chore: forge fmt * test: fix fork tests
* test: add unit tests for consolidation and withdrawal
* chore: make bindings * chore: misc feedback
…bility (#1356) * docs: update release README * chore: fix eigenpod version in 1.6.0 script
**Motivation:** add semver to eigen **Modifications:** add semver to eigen **Result:** add semver to eigen
have to resolve a ton of conflicts when rebasing to main after redistribution merged to preserve |
c2934a5
to
555aef9
Compare
commits in changelog will be added after merging to main, otherwise we will lose commit of this pr |
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.
LGTM
Motivation:
Modifications:
1. Add 4 methods to
EigenPod.sol
:See full method docs in
IEigenPod.sol
- they're long, so I didn't copy them here 😄2. Add 4 events to
EigenPod.sol
:3. Modify 2
EigenPod
methods:EigenPod._updateCheckpoint()
:currentCheckpointTimestamp == 0
EigenPod.currentCheckpoint()
.EigenPod.GENESIS_TIME()
:EigenPods
.4. Modify 5 events in
EigenPod.sol
:pubkeyHash
when referencing a validator.Updated event definitions:
5. Add 2 events to
Eigen.sol
:Result:
EigenPods will support Pectra features, allowing the pod owner or proof submitter of a pod to initiate consolidations and withdrawals via the EIP-7002 and EIP-7521 predeploys.