-
Notifications
You must be signed in to change notification settings - Fork 580
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: make 09-localhost stateless #6683
Merged
colin-axner
merged 26 commits into
main
from
gjermund/5959-make-localhost-a-stateless-implementation
Jul 3, 2024
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
614fb22
Remove client state from localhost light client
gjermundgaraba 952423f
Initial store upgrade for v9 localhost statelessness
gjermundgaraba cc1fa35
Merge branch 'main' into gjermund/5959-make-localhost-a-stateless-imp…
gjermundgaraba fe6519f
Remove localhost creation and self state proof
gjermundgaraba 249518b
Wire up migrations
gjermundgaraba 251825d
Update docs for 09-localhost for 09-localhost
gjermundgaraba 465a80c
Merge branch 'main' into gjermund/5959-make-localhost-a-stateless-imp…
gjermundgaraba bb92f22
lint
gjermundgaraba 93082f5
Merge branch 'main' into gjermund/5959-make-localhost-a-stateless-imp…
gjermundgaraba 0847cbe
Merge branch 'main' into gjermund/5959-make-localhost-a-stateless-imp…
gjermundgaraba f573061
Merge branch 'main' into gjermund/5959-make-localhost-a-stateless-imp…
crodriguezvega 84c3236
refactor: simplify migrations, remove unnecessary iteration over clie…
colin-axner ec25710
refactor: remove localhost client state from queries
colin-axner c207393
refactor: remove localhost client state, condense localhost impl into…
colin-axner 8321f41
rename: MigrateToStatelessLocalhost
colin-axner c58f93a
lint
colin-axner 261231e
Merge branch 'main' of github.com:cosmos/ibc-go into gjermund/5959-ma…
colin-axner 58ca2d9
Update docs/docs/05-migrations/13-v8-to-v9.md
colin-axner ab99084
Update docs/docs/05-migrations/13-v8-to-v9.md
colin-axner 61f8ea6
review: docs + linting cleanups
colin-axner 79b809f
Merge branch 'gjermund/5959-make-localhost-a-stateless-implementation…
colin-axner c107950
Merge branch 'main' of github.com:cosmos/ibc-go into gjermund/5959-ma…
colin-axner fdd7806
refactor: remove unused func
colin-axner a66ccff
Update modules/core/02-client/keeper/migrations.go
colin-axner 6028aa0
markdown lint
colin-axner e00c792
Merge branch 'gjermund/5959-make-localhost-a-stateless-implementation…
colin-axner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
grpc should no longer return localhost in its query. There's no information to obtain, as it would just return back the height of the request
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.
that makes sense!