Open assafmo opened 5 months ago
[!NOTE]
Reviews paused
Use the following commands to manage reviews:
@coderabbitai resume
to resume automatic reviews.@coderabbitai review
to trigger a single review.
The changes introduce a Token Factory module into the application, enhancing its functionality for managing tokens. Additionally, a comprehensive configuration for deploying a Hyperlane application is established, detailing services, network settings, and validator configurations. The integration of TIA as a gas token is emphasized, with updates to documentation and configuration files to support its use in transactions across multiple blockchain networks.
File Path | Change Summary |
---|---|
guides/assets/hyperlane-use-tia-for-gas/*.diff |
Integration of Token Factory module, including updates to WasmApp , permissions, and module registration. |
guides/assets/hyperlane-use-tia-for-gas/docker-compose.yml |
Defines a multi-service architecture for a Hyperlane application with specific configurations for each service. |
guides/assets/hyperlane-use-tia-for-gas/hyperlane-config-2.yaml |
New YAML configuration for a multi-signature governance model, detailing network settings and hooks. |
guides/assets/hyperlane-use-tia-for-gas/hyperlane-config.yaml |
Configuration for blockchain networks, including deployment settings for an Interchain Security Model. |
guides/assets/hyperlane-use-tia-for-gas/relayer.json |
Configuration for a relayer service, defining relay chains and whitelist for cross-domain interactions. |
guides/assets/hyperlane-use-tia-for-gas/stride-internal-1.json |
JSON object defining artifacts related to the Hyperlane protocol, including unique identifiers. |
guides/assets/hyperlane-use-tia-for-gas/utia-localwasm.json |
Configuration for a bridged native asset, specifying ownership and denomination. |
guides/assets/hyperlane-use-tia-for-gas/utia-stride.json |
Configuration for a collateral native asset, detailing ownership and IBC token references. |
guides/assets/hyperlane-use-tia-for-gas/validator.localwasm.json |
Validator configuration for the localwasm chain, including database and checkpoint syncer settings. |
guides/assets/hyperlane-use-tia-for-gas/validator.strideinternal1.json |
Validator configuration for the strideinternal1 chain, outlining database and synchronization methods. |
guides/assets/hyperlane-use-tia-for-gas/wasmd.Dockerfile |
Dockerfile for building the wasmd application with necessary tools and configurations. |
guides/hyperlane-use-tia-for-gas.md |
Guide on integrating Hyperlane as a gas token mechanism, detailing setup and deployment procedures. |
guides/use-tia-for-gas.md |
Updated document with trust assumptions and gas price changes related to TIA usage. |
tutorials/cosmwasm.md |
Grammatical corrections and formatting adjustments to improve readability. |
š In the meadow where tokens play,
A factory blooms, brightening the day.
With TIA as gas, we hop and cheer,
Cross-chain journeys, adventures near!
Docker and YAML, all set to go,
In Hyperlane's dance, we joyfully flow! š¼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@assafmo what's missing? when ready for review?
Still on it. Will tag you when it's ready.
@assafmo, when do you plan to finish it?
Hey guys, this is ready for a review
please review and resolve Coderabbit suggestions as you see fit @assafmo ! some are junk and can definitely be thrown out, but some may be useful.
@coderabbitai ignore
@coderabbitai ignore
The tutorial should inline the assets. Otherwise the tutortial is at risk of breaking without warning due to the
wget
s targeting code not on the docs website.
Yeah, there are a lot of new configs that we introduce here, and initially we had it inline but decided to do it using wget because the entire guide was becoming echo "$HUGE_CONFIG" > config.json
.
Lmk if you want me to restore the old way, keep the wget way, or do anything else. We'll fix all the code review comments in the next few days.
The tutorial should inline the assets. Otherwise the tutortial is at risk of breaking without warning due to the
wget
s targeting code not on the docs website.Yeah, there are a lot of new configs that we introduce here, and initially we had it inline but decided to do it using wget because the entire guide was becoming
echo "$HUGE_CONFIG" > config.json
.Lmk if you want me to restore the old way, keep the wget way, or do anything else. We'll fix all the code review comments in the next few days.
I think all the assets should live on the branch of the fork I recommended. i.e. Stride-Labs/wasmd
.
Then the first step of cloning that repo and checking out the rollkit branch handles all assets and code files.
Additionally this could keep everything in one place for updates and testing in the future.
Mainly add "Use Hyperlane TIA as gas token" as an alternative to IBC guide, but also:
guides/use-tia-for-gas.md
tutorials/cosmwasm.md
(and add--depth 1
to all thegit clone
s)Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Documentation