Skip to content
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

Minimize the amount that is pulled from upstream #1527

Open
evan-forbes opened this issue Nov 13, 2024 · 0 comments
Open

Minimize the amount that is pulled from upstream #1527

evan-forbes opened this issue Nov 13, 2024 · 0 comments
Labels
T:dependencies Type: Pull requests that update a dependency file WS: Maintenance 🔧

Comments

@evan-forbes
Copy link
Member

As we make more and more changes here, it stops making sense to merge changes from upstream in this fork. Instead of merging changes from upstream, which takes a long time to merge and is essentially impossible to review, we could instead only pull in the changes that matter for compatability with things that we care about.

This includes compatability with the sdk and IBC.

IBC likely works as is with a replace statement since there haven't been breaking changes to the proto files (that would mean IBC v6 would be unable to verify a v8)

The cosmos-sdk would require updates to interfaces, although we don't need to fully support those new methods unless we want to use them.

@evan-forbes evan-forbes added T:dependencies Type: Pull requests that update a dependency file WS: Maintenance 🔧 labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T:dependencies Type: Pull requests that update a dependency file WS: Maintenance 🔧
Projects
None yet
Development

No branches or pull requests

1 participant