-
Notifications
You must be signed in to change notification settings - Fork 1
remove the option for standard building #38
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
Conversation
4933c58 to
a49c1d9
Compare
818857d to
8e0c207
Compare
There was a problem hiding this 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!
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
8e0c207 to
3dada55
Compare
We'll only be running with flashblocks