Skip to content

Conversation

@mmsqe
Copy link
Contributor

@mmsqe mmsqe commented Nov 17, 2025

Description

Closes: #25564, for test info

@github-actions github-actions bot added the C:CLI label Nov 17, 2025
@codecov
Copy link

codecov bot commented Nov 17, 2025

Codecov Report

❌ Patch coverage is 49.12281% with 58 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.29%. Comparing base (aff4fe4) to head (06da98e).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
server/grpc/server.go 7.50% 37 Missing ⚠️
client/grpc_query.go 75.00% 10 Missing ⚠️
server/start.go 12.50% 7 Missing ⚠️
client/context.go 42.85% 4 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #25565      +/-   ##
==========================================
+ Coverage   68.39%   70.29%   +1.90%     
==========================================
  Files         856      821      -35     
  Lines       55773    53780    -1993     
==========================================
- Hits        38144    37807     -337     
+ Misses      17629    15973    -1656     
Files with missing lines Coverage Δ
server/config/config.go 93.10% <100.00%> (+1.43%) ⬆️
server/config/toml.go 81.81% <ø> (ø)
testutil/network/util.go 87.90% <100.00%> (+0.19%) ⬆️
client/context.go 81.34% <42.85%> (+0.16%) ⬆️
server/start.go 19.13% <12.50%> (+0.08%) ⬆️
client/grpc_query.go 78.57% <75.00%> (+2.20%) ⬆️
server/grpc/server.go 44.30% <7.50%> (-38.63%) ⬇️

... and 39 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mmsqe mmsqe marked this pull request as ready for review November 17, 2025 14:16
Copy link
Contributor

@aljo242 aljo242 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this looks good to me - mainly just wanting a bit more documentation and testing :)

Copy link
Contributor

@aljo242 aljo242 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with just tiny nits

@aljo242 aljo242 enabled auto-merge November 19, 2025 15:15
@aljo242 aljo242 added this pull request to the merge queue Nov 19, 2025
Merged via the queue into cosmos:main with commit f4e2ce0 Nov 19, 2025
40 of 41 checks passed
mergify bot pushed a commit that referenced this pull request Nov 19, 2025
)

Co-authored-by: Alex | Cosmos Labs <[email protected]>
(cherry picked from commit f4e2ce0)

# Conflicts:
#	server/start.go
warpbuild-benchmark-bot bot added a commit to WarpBuilds/cosmos-sdk that referenced this pull request Nov 19, 2025
aljo242 added a commit that referenced this pull request Nov 19, 2025
mmsqe added a commit to mmsqe/cosmos-sdk that referenced this pull request Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: support for multi gRPC query clients serve with old binary

2 participants