-
Notifications
You must be signed in to change notification settings - Fork 1
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
[WIP]: WARP #65
Draft
ramilexe
wants to merge
7
commits into
main
Choose a base branch
from
warp
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
[WIP]: WARP #65
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* warp message: addMessage * introduce codec, codec manager; add unit tests; add all necessary utilities in MVP * implement bls signature methods; add test cases * get rid of avalanchego dependency; use cometbft instead * delete unused variable * lint go imports * lint go imports --------- Co-authored-by: Ivan Sukach <ivansukach@github.com>
Co-authored-by: Ivan Sukach <ivansukach@github.com>
* preliminary * prelimiry: get message signature * add codec version to packer of codec * process error of codec type registration * add exception for static check * ignore U1000 linter unused function * get rid of unnecesary slicing, set space in comment * linter: sort imports --------- Co-authored-by: Ivan Sukach <ivansukach@github.com>
* move warpSigner backend functionality to signMessage backend method * get rid of unnecessary slicing --------- Co-authored-by: DESKTOP-765JFGJ\Admin <ivan.sukach1@ibm.com>
* WIP: prelimiry warp service * warp service vm integration * warp get message test * warp RPC: get message signature implementation * get rid of RPC layer for warp service * fix staticcheck warnings * goimport, add space between space and slash --------- Co-authored-by: DESKTOP-765JFGJ\Admin <ivan.sukach1@ibm.com> Co-authored-by: Ivan Sukach <ivansukach@github.com>
* preliminary version of warp aggregate signature functionality * aggregate signature: signature getter, aggregator, message * panic implementation of peer network client * simplify validator set receiving * get rid of client field, set signature getter instead * signature getter: replace peer.NetworkSignatureGetter with apiFetcher * get rid of unnecessary functionality * fix sprintf template for string representation of message type * add exceptions for linter * get rid of field index of Validator struct * move comment to another position * nolint:SA1019 * change format of linter exceptional comment * unslice; goimports; add G507 exception * setup exception comment for linter * setup #nosec exception comment for linter * disable G507 check * get rid of import deprecated ripemd160 library --------- Co-authored-by: Ivan Sukach <ivansukach@github.com>
* AddMessage RPC method * change error template * goimports warp_service --------- Co-authored-by: Ivan Sukach <ivansukach@github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.