Skip to content

Comments

cip(draft): Hyperlane#256

Closed
S1nus wants to merge 5 commits intocelestiaorg:mainfrom
S1nus:c/hyperlane
Closed

cip(draft): Hyperlane#256
S1nus wants to merge 5 commits intocelestiaorg:mainfrom
S1nus:c/hyperlane

Conversation

@S1nus
Copy link
Contributor

@S1nus S1nus commented Feb 21, 2025

Overview

A new CIP, to add Hyperlane to celestia-app.

@rootulp rootulp marked this pull request as draft February 21, 2025 11:06
@rootulp
Copy link
Collaborator

rootulp commented Feb 21, 2025

Converting to draft b/c doesn't seem ready for review.

@jcstein
Copy link
Member

jcstein commented Feb 21, 2025

Please use a filename such as cip-add-hyperlane.md instead of assigning a number. See the example: https://cips.celestia.org/cip-template.html

@S1nus
Copy link
Contributor Author

S1nus commented Feb 21, 2025

Please use a filename such as cip-add-hyperlane.md instead of assigning a number. See the example: https://cips.celestia.org/cip-template.html

done


## Test Cases

TBD
Copy link
Collaborator

Choose a reason for hiding this comment

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

Proposal: add a test case that entails deploying Hyperlane on a Celestia testnet and establish a Hyperlane bridge from another Cosmos / EVM testnet to Celestia testnet. Then send TIA from Celestia to remote chain and verify it works. Then send a non TIA token from remote chain to Celestia and verify it doesn't work. Note: this assumes that we add something like the tokenfilter to the Hyperlane Cosmos SDK module on Celestia.

@cmwaters
Copy link
Collaborator

cmwaters commented Mar 6, 2025

I'm unable to assign myself to this, but I'm going to take over from @S1nus to try get this CIP merged

@rootulp rootulp assigned cmwaters and unassigned S1nus Mar 6, 2025
@S1nus
Copy link
Contributor Author

S1nus commented Mar 6, 2025

I'm unable to assign myself to this, but I'm going to take over from @S1nus to try get this CIP merged

Feel free to fork / take it over and we can close this

@jcstein
Copy link
Member

jcstein commented Mar 6, 2025

closing in favor of #256

@jcstein jcstein closed this Mar 6, 2025
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.

4 participants