-
Notifications
You must be signed in to change notification settings - Fork 910
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
core: package managing Core Node #51
Conversation
…t with configuration
…work with HTTP by default
… implemntation; update tests accordingly
…estutils: move Core mock to core package
Note that the linter fails after some weird infrastructure reason. #53 |
Okay so decision here (as per #52 )is that we will offer both |
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.
Rene's suggestion makes sense. Otherwise, PR looks good to me.
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
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.
Still waiting on #55 and some comments to be addressed, but otherwise good with this right now.
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
* embed node add tests * clean up tests + rename * fix doc
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
Co-authored-by: rene <41963722+renaynay@users.noreply.github.com>
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.
ship 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.
The PR introduces a core package that mainly provides different Clients to work with Core Node. It introduces embeddable Core Node and keeps existing RPC clients. It also makes embedding Core a default behavior for Full Node with configurability to connect to remote Core if requested.
Closes #50