cosmos / ibc-proto-rs

Rust Protobuf definitions and gRPC clients for interacting with Cosmos SDK, IBC and Interchain Security
Apache License 2.0
42 stars 25 forks source link

Should I abandon `cosmos-sdk-proto` and point people at `ibc-proto` instead? #182

Closed tony-iqlusion closed 6 months ago

tony-iqlusion commented 6 months ago

The two projects duplicate each others' functionality, but ibc-proto seems more mature at this point.

I find the whole situation a bit confusing and frustrating: ibc-proto has a much bigger scope than just IBC, and is instead a full Cosmos SDK Protobuf library.

cosmos-sdk-proto also has more downloads and a "Used by" count of 661 (whereas that number isn't populated for ibc-proto-rs for whatever reason). Just by virtue of its name it's more discoverable.

But I find duplication of effort to be incredibly demotivating.

romac commented 6 months ago

Hi Tony! Thanks for starting this discussion, I absolutely agree that the current situation is really subpar and we should aim to improve it.

How about I spend some time this week investigating whether we can deduplicate work by:

Would that seem like a good course of action to you? Is there something else you think should be done to remediate the current situation?

tony-iqlusion commented 6 months ago

That approach seems fine to me.

I think ideally they'd be in the same repo to synchronize changes, but if that's the direction you'd like to go, I'm happy to work toward that in whatever manner.

tony-iqlusion commented 6 months ago

With #187 open I'll go ahead and close this

romac commented 6 months ago

👍 Shall I open a PR on cosmos-rust so we can discuss there how best to proceed with all this?

tony-iqlusion commented 6 months ago

Yep, absolutely