-
Notifications
You must be signed in to change notification settings - Fork 40
docs: update hyperlane cli #1264
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
docs: update hyperlane cli #1264
Conversation
✅ Deploy Preview for oasisprotocol-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Please also test this and ensure it works, once oasisprotocol/oasis-sdk#2215 is lve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice. Since we can use standard Hyperlane tooling, I would:
- Move the Introduction+Hyperlane Core deployment sections to the Hyperlane top chapter somewhere at the end.
- Remove the Hyperlane CLI chapter (+redirect it to hyperlane/)
As @ptrus mentioned, before merging this please wait until a sapphire-testnet version will support public storage slots and test it first.
This PR affects #1239 |
This is now also live on mainnet. |
@rube-de Can you check it again, if the hyperlane CLI now works out of the box and merge this? |
e3a00a9
to
6c90304
Compare
tested, see: oasisprotocol/sapphire-paratime#588 |
6c90304
to
69235b9
Compare
Simplifies the Hyperlane CLI docs by removing Sapphire-specific instructions and switching to the standard CLI installation flow. Updates introduction and usage examples for clarity.
Merge once Sapphire with oasisprotocol/oasis-sdk#2215 is live.
Preview