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.

Sign up for GitHub

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

Jump to bottom

Confirmation rule prerequisite #6257

Closed
twoeths opened this issue Jan 8, 2024 · 1 comment · Fixed by #6288
Closed

Confirmation rule prerequisite #6257

twoeths opened this issue Jan 8, 2024 · 1 comment · Fixed by #6288
Assignees
@twoeths
Labels
meta-feature-request Issues to track feature requests. spec-deneb 🐡 Issues targeting the Deneb spec version

Comments

@twoeths
Copy link
Contributor

twoeths commented Jan 8, 2024

Problem description

We need some forkchoice changes for the safe execution of the confirmation rule, here's the main PR ethereum/consensus-specs#3339

Solution description

specified in ethereum/consensus-specs#3431

Additional context

No response
@twoeths twoeths added the meta-feature-request Issues to track feature requests. label Jan 8, 2024
@twoeths twoeths self-assigned this Jan 9, 2024
@philknows philknows added the spec-deneb 🐡 Issues targeting the Deneb spec version label Jan 11, 2024
@g11tech g11tech mentioned this issue Jan 11, 2024
Closed
40 tasks
@philknows
Copy link
Member

philknows commented Jan 11, 2024 •
edited
Loading

Consensus spec PR will get merged shortly with spec release around Monday. We need to get this out for mainnet fork of Deneb @tuyennhv. Mainnet fork date bit of a moving target depending on testnet performance.

@twoeths twoeths mentioned this issue Jan 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@twoeths twoeths

Labels
meta-feature-request Issues to track feature requests. spec-deneb 🐡 Issues targeting the Deneb spec version
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: forkchoice filter change ChainSafe/lodestar
2 participants
@twoeths @philknows