Skip to content

Add add_validator_to_registry fn#3311

Merged
hwwhww merged 3 commits intoethereum:devfrom
dapplion:add_validator_to_registry
Jun 16, 2023
Merged

Add add_validator_to_registry fn#3311
hwwhww merged 3 commits intoethereum:devfrom
dapplion:add_validator_to_registry

Conversation

@dapplion
Copy link
Member

Suggested in #3307 (comment) by @mkalinin allows the altair future forks spec to be leaner when adding new state fields that require per validator initialization

@haxxpop
Copy link
Member

haxxpop commented Apr 3, 2023

I think this is a great work. This makes the spec leaner, so I'm in favor of this.

@djrtwo djrtwo mentioned this pull request Apr 19, 2023
@dapplion dapplion force-pushed the add_validator_to_registry branch from 1702586 to 680b026 Compare May 9, 2023 10:22
Copy link
Contributor

@hwwhww hwwhww left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @dapplion!

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.

3 participants