Skip to content

Conversation

@akundaz
Copy link
Collaborator

@akundaz akundaz commented Nov 3, 2025

We'll only be running with flashblocks

@akundaz akundaz self-assigned this Nov 3, 2025
@akundaz akundaz force-pushed the ak-remove-standard-building branch from 4933c58 to a49c1d9 Compare November 3, 2025 20:47
@akundaz akundaz force-pushed the ak-remove-standard-building branch 2 times, most recently from 818857d to 8e0c207 Compare November 5, 2025 15:22
@akundaz akundaz requested a review from julio4 November 6, 2025 16:12
Copy link
Collaborator

@julio4 julio4 left a comment

Choose a reason for hiding this comment

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

Just ensure that it don't adds new failing tests and seems gtg!

Copy link
Collaborator

Choose a reason for hiding this comment

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

would it be possible/wise to puts the ws_addr inside the LocalNode<Flashblocks, _> with a getter like node.ws_addr()?

To do like:

let node = Flashblocks::test_node().await?;
let ws_addr = node.ws_addr();

its not a big deal so no need to actually implement it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i'll get around to this in another pr since we don't even have a node type so this will take a bit of code

@akundaz akundaz force-pushed the ak-remove-standard-building branch from 8e0c207 to 3dada55 Compare November 10, 2025 18:53
@akundaz akundaz merged commit e7b43b6 into main Nov 10, 2025
3 of 4 checks passed
@akundaz akundaz deleted the ak-remove-standard-building branch November 10, 2025 18:58
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.

2 participants