Skip to content

Conversation

@bnkamalesh
Copy link
Member

[minor] added helpers for GRPC status codes and messages
[minor] introduced 3 new error types NotImplemented, ContextCanceled, ContextTimedout
[minor] context errors are automatically identified in error types
[patch] improved test coverage
[-] updated benchmark results

[minor] introduced 3 new error types NotImplemented, ContextCanceled, ContextTimedout
[minor] context errors are automatically identified in error types
[patch] improved test coverage
[-]     updated benchmark results
@bnkamalesh bnkamalesh self-assigned this Jul 3, 2025
@bnkamalesh bnkamalesh added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 3, 2025
@bnkamalesh bnkamalesh enabled auto-merge (squash) July 3, 2025 09:21
@coveralls
Copy link

coveralls commented Jul 3, 2025

Pull Request Test Coverage Report for Build 16046726488

Details

  • 187 of 214 (87.38%) changed or added relevant lines in 4 files are covered.
  • 4 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+6.9%) to 80.556%

Changes Missing Coverage Covered Lines Changed/Added Lines %
errors.go 1 3 33.33%
grpc.go 72 74 97.3%
helper.go 40 63 63.49%
Files with Coverage Reduction New Missed Lines %
errors.go 2 83.28%
helper.go 2 68.01%
Totals Coverage Status
Change from base Build 11291483275: 6.9%
Covered Lines: 609
Relevant Lines: 756

💛 - Coveralls

@bnkamalesh bnkamalesh disabled auto-merge July 3, 2025 09:31
@bnkamalesh bnkamalesh enabled auto-merge (squash) July 3, 2025 09:32
@bnkamalesh bnkamalesh disabled auto-merge July 3, 2025 09:32
@bnkamalesh bnkamalesh merged commit c38d0fb into master Jul 3, 2025
1 check passed
@bnkamalesh bnkamalesh deleted the grpcerrors branch July 3, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants