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

cosmos-sdk-proto: Remove IBC protos in favour of the ibc-proto crate #495

Merged
merged 5 commits into from
Aug 15, 2024

Conversation

romac
Copy link
Member

@romac romac commented Aug 15, 2024

This is a new version of #459.

Best reviewed commit by commit.

Changes

  • Remove ibc-go submodule and the corresponding generated proto definitions
  • Re-export ICS 23 proto definitions from the ics23 crate instead of bundling them in cosmos-sdk-proto

@@ -361,9 +298,6 @@ fn copy_and_patch(src: impl AsRef<Path>, dest: impl AsRef<Path>) -> io::Result<(
"/// Generated server implementations.\n\
#[cfg(feature = \"grpc\")]",
),
// Use `tendermint_proto` as source of `google.protobuf` types
// TODO(tarcieri): figure out what's wrong with our `buf` config and do it there
("::prost_types::", "::tendermint_proto::google::protobuf::"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh hey, thanks for cleaning this up, I forgot to remove it when we fixed the buf config

Copy link
Member

@tony-iqlusion tony-iqlusion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, this looks good to me. I just cut a release of cosmos-sdk-proto so I don't know how you want to go about merging/releasing these changes.

@tony-iqlusion
Copy link
Member

tony-iqlusion commented Aug 15, 2024

I could merge this and cut a cosmos-sdk-proto 0.25.0-pre prerelease

@romac
Copy link
Member Author

romac commented Aug 15, 2024

I could merge this and cut a cosmos-sdk-proto 0.25.0-pre prerelease

Sounds good!

@tony-iqlusion
Copy link
Member

Alright, let's do it

@tony-iqlusion tony-iqlusion merged commit fb341da into cosmos:main Aug 15, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants