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.

Sign up for GitHub

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

Jump to bottom

Hermes CLI for channel create #731

Closed
wants to merge 16 commits into from
Closed

Hermes CLI for channel create #731

wants to merge 16 commits into from

Conversation

adizere
Copy link
Member

@adizere adizere commented Mar 10, 2021

Closes: #715

Description

For contributor use:

  • Updated the Unreleased section of CHANGELOG.md with the issue.
  • If applicable: Unit tests written, added test to CI.
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Updated relevant documentation (docs/) and code comments.
  • Re-reviewed Files changed in the Github PR explorer.

adizere added 16 commits March 10, 2021 09:30
@adizere
Added domain type def.
7739e43 
Also cleaned-up the documentation, which looked very sloppy.
https://docs.rs/ibc/0.1.1/ibc/ics02_client/msgs/index.html
@adizere
Added partial handler & command
617c51b
@adizere
Added upgrade proto files. Added Cargo.lock for proto-compiler
402409e
@adizere
Prep for query_upgraded_client_state
4bc233e
@adizere
Method & support for querying ugpraded client state
d5e7518
@adizere
Support for querying the upgraded consensus state
56231dd
@adizere
Minor dev scripts enhancements & bugs
a0a41b5
@adizere
Added guide. Uses patched Go relayer
1fe7044
@adizere
Proto conversion for upgrade msg. Refactored upgrade() impl
ba199bc
@adizere
Fix missing signer bug
a8af82f
@adizere
Update msg bf. upgrade. Event parsing
1217b12
@adizere
Changelog. Revised guide
070a408
@adizere
Aesthetic nits based on file review
078a6aa
@adizere
Clarifications in the test instructions
44ee3a2
@adizere
Added channel handshake basic skeleton
ca71d84
@adizere
Added alternative invocation
78d6edb
@adizere adizere changed the base branch from master to adi/357_upgrade March 10, 2021 10:15
@adizere adizere changed the base branch from adi/357_upgrade to master March 10, 2021 10:16
@adizere
Copy link
Member Author

adizere commented Mar 10, 2021

Abandoned in favor of a cleaner approach in #732

@adizere adizere closed this Mar 10, 2021
@adizere adizere deleted the adi/715_chan branch March 10, 2021 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Channel create CLI
1 participant
@adizere