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

feat: add interchain security (ICS) support to v8 branch #1041

Closed
jtieri opened this issue Apr 1, 2024 · 4 comments · Fixed by #1088
Closed

feat: add interchain security (ICS) support to v8 branch #1041

jtieri opened this issue Apr 1, 2024 · 4 comments · Fixed by #1088
Assignees
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request good first issue Good for newcomers

Comments

@jtieri
Copy link
Contributor

jtieri commented Apr 1, 2024

The Interchain Security team recently stated that ICS would support SDK v0.50.x and ibc-go v8.x.x which means that we will soon be able to bring support for ICS into the main branch (v8).

@jtieri jtieri added enhancement New feature or request good first issue Good for newcomers dependencies Pull requests that update a dependency file labels Apr 1, 2024
@Reecepbcups
Copy link
Member

Reecepbcups commented Apr 3, 2024

also ref: backport #989 (local-ic ICS support) -> v8

@Reecepbcups
Copy link
Member

Reecepbcups commented Apr 23, 2024

Branch is up and alpha v5.0.0-alpha.1 tagged -> https://github.com/cosmos/interchain-security/tree/release/v5.x

Will need to re-add support to hermes as well. ref: #1067

@MSalopek
Copy link

MSalopek commented Apr 30, 2024

Thank you for the effort! I was not aware of this until I saw a github mention.

This is the latest ICS v5: https://github.com/cosmos/interchain-security/releases/tag/v5.0.0-rc0

The v5.0.0-rc0 has feature parity with v4.1.0 (has epochs code). After some docs changes a full release willl be tagged.

Can I help you with anything here?

@Reecepbcups
Copy link
Member

Reecepbcups commented Apr 30, 2024

@MSalopek Thanks! It has already been integrated on our side on the latest commit at the time: #1088

Just pending review on our side to get into the next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants