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.

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

chore: make NodeEvent generic over NodePrimitives #13534

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Conversation

klkvr
Copy link
Collaborator

@klkvr klkvr commented Dec 23, 2024

Based on #13533

We need to propagate NodePrimitives to BeaconConsensusEngineEvent

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

right we def need this

@@ -350,12 +353,12 @@ struct CurrentStage {
}

/// A node event.
#[derive(Debug)]
pub enum NodeEvent {
#[derive(Debug, derive_more::From)]
Copy link
Collaborator

Choose a reason for hiding this comment

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

nice driveby

@mattsse mattsse added A-observability Related to tracing, metrics, logs and other observability tools A-sdk Related to reth's use as a library labels Dec 23, 2024
Base automatically changed from klkvr/move-engine-sender to main December 23, 2024 23:09
@klkvr klkvr requested a review from rkrasiuk as a code owner December 23, 2024 23:09
@klkvr klkvr force-pushed the klkvr/generic-node-event branch from 1c4f128 to 82837e6 Compare December 23, 2024 23:12
@klkvr klkvr enabled auto-merge December 23, 2024 23:13
@klkvr klkvr force-pushed the klkvr/generic-node-event branch from 82837e6 to c11f8a8 Compare December 23, 2024 23:18
@klkvr klkvr disabled auto-merge December 23, 2024 23:21
@klkvr klkvr enabled auto-merge December 23, 2024 23:21
@klkvr klkvr added this pull request to the merge queue Dec 23, 2024
Merged via the queue into main with commit 934fd1f Dec 23, 2024
43 checks passed
@klkvr klkvr deleted the klkvr/generic-node-event branch December 23, 2024 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-observability Related to tracing, metrics, logs and other observability tools A-sdk Related to reth's use as a library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants