Skip to content

Expose gRPC call status - #11

Merged
samuel-williams-shopify merged 4 commits into
mainfrom
call-status
Aug 21, 2026
Merged

Expose gRPC call status#11
samuel-williams-shopify merged 4 commits into
mainfrom
call-status

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add Protocol::GRPC::Call#status
  • read grpc-status directly from response headers
  • return nil when no response or status is available
  • avoid changing the response header policy
  • document the new nullable API

Rebased onto main after removal of the unused Call cancellation state in #12.

Verification

  • 217 tests passing
  • RuboCop: 56 files, no offenses
  • documentation coverage: 98/98 public definitions

@samuel-williams-shopify
samuel-williams-shopify merged commit f7deeca into main Aug 21, 2026
40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the call-status branch August 21, 2026 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant