-
Notifications
You must be signed in to change notification settings - Fork 17
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
chore!: remove merkle library #91
Conversation
This reverts commit fdf1e92.
Shoot, we haven't decided what we do instead. |
We're actually not using this library atm (we're still depending on the Actually I'm wrong. We do use it so would have to revert back to |
Note: there are usages of go-square/merkle in celestia-app (e.g. here) so I'm guessing those usages will need to be updated to celestia-core |
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.
Seems fine to me. In go-square v2 release notes we should provide guidance to any consumers who use go-square/merkle to migrate to something else (potentially celestia-core merkle)
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.
yay
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.
Actually I'm wrong. We do use it so would have to revert back to celestia-core
Just saw the edit. I am strictly against go-square depending on celestia-core, even if the merkle pkg is a separate go.mod inside
go-square doesn't need to depend on celestia-core b/c nothing breaks if we remove the |
What does revert back mean? |
I think consumers of go-square/merkle (i.e. celestia-app) would have to revert back to consuming it from celestia-core. |
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 see, I misunderstood the message then
This reverts commit fdf1e92.
Closes: #39