-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Migrate missing x/distribution
CLI queries to proto
#7244
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7244 +/- ##
=======================================
Coverage 54.91% 54.92%
=======================================
Files 575 575
Lines 39536 39544 +8
=======================================
+ Hits 21712 21718 +6
- Misses 16031 16032 +1
- Partials 1793 1794 +1 |
This pull request introduces 1 alert when merging 28c8997 into b52ffa0 - view on LGTM.com new alerts:
|
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.
utACK
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.
lgtm
Description
ref: #6987
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes