Skip to content

Conversation

@danyalprout
Copy link
Collaborator

Fixes #316

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 13, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@danyalprout danyalprout added the F-do-not-merge Flag: don't merge until this label is removed. label Jan 13, 2026
@danyalprout danyalprout marked this pull request as ready for review January 14, 2026 01:09
@danyalprout danyalprout requested a review from refcell January 14, 2026 01:17
@danyalprout danyalprout enabled auto-merge January 14, 2026 01:33
Comment on lines +72 to +81
let mut canonical_stream =
BroadcastStream::new(ctx.provider().subscribe_to_canonical_state());
tokio::spawn(async move {
while let Some(Ok(notification)) = canonical_stream.next().await {
let committed = notification.committed();
for block in committed.blocks_iter() {
state_for_canonical.on_canonical_block_received(block.clone());
}
}
});
Copy link
Contributor

Choose a reason for hiding this comment

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

wow this is nice

Copy link
Contributor

@refcell refcell left a comment

Choose a reason for hiding this comment

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

Great PR

@danyalprout danyalprout added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 69fdea5 Jan 14, 2026
18 checks passed
@danyalprout danyalprout deleted the fix-exex branch January 14, 2026 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

F-do-not-merge Flag: don't merge until this label is removed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(flashblocks): ensure flashblocks exex does not lag

4 participants