Skip to content

Conversation

sampajano
Copy link
Collaborator

@sampajano sampajano commented Nov 11, 2021

This complements #1164 to ensure the properties of RpcError are exported as well, and add tests.

(btw even without the @export annotation these 2 fields are exported today, but that's somewhat by accident -- it's due to them being referred by other classed rather than guaranteed.)

@sampajano sampajano changed the title Fixing missing exports from RpcError and add test. Add missing exports from RpcError and add test. Nov 11, 2021
Copy link
Collaborator

@stanley-cheung stanley-cheung left a comment

Choose a reason for hiding this comment

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

LGTM, very subtle!

@sampajano sampajano merged commit 1efe741 into grpc:master Nov 12, 2021
@sampajano sampajano deleted the 1-rpcerror-export branch January 29, 2022 06:08
@sampajano sampajano changed the title Add missing exports from RpcError and add test. Add missing exports from RpcError and add test. Jan 29, 2022
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.

2 participants