-
Notifications
You must be signed in to change notification settings - Fork 278
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
Update to the lazyledger packages instead of tendermint and the cosmos-sdk #7
Conversation
Looks good to me! Wondering if we shouldn't merge in the CI changes (#8) first? |
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.
LGTM.
Is it tested that the starport scaffolding #1
is working correctly after the whitespace changes?
Do you mind elaborating a bit, @tzdybal? I'm not quite sure what This branch has been tested to start a new chain, but it does not use starport for that. If you're talking about adding further scaffolding, then the answer is that any additions will have to be patched similarly to what was done to this branch before they could be added. |
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.
Needs resolving the merge conflicts / rebasing onto master. If CI passes afterwards, I'd approve :-)
080f261
to
6f16b81
Compare
I'm still trying to figure out the last |
072f7a2
to
18fdb67
Compare
switch to using lazyledger-core and the fork of the sdk update to the latest verison of lazyledger-core use the latest version of lazyledger-core and the lazyledger fork of the sdk gofmt gofmt ci fix: add the make proto-lint command switch to the lazyledger-core proto files remove redundant message types
update from deprecated cli command add version to buf.yaml change lazyledger-app proto package declaration in accordance with buf linting
18fdb67
to
644d2d2
Compare
Ok, now we're good to go. The only CI failure is because we don't have any tests. |
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.
Thanks Evan! LGTM 👍
Uses the replace directive to use the latest LazyLedger fork of the cosmos-sdk, along with manually replacing tendermint with lazyledger-core
Closes #4