Skip to content
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

grpc-native-core: Update CallOptions type to allow custom options (#433) #437

Merged
merged 1 commit into from
Jul 17, 2018

Conversation

TLadd
Copy link
Contributor

@TLadd TLadd commented Jul 13, 2018

Allow custom options as a means of passing data per call to client interceptors

Resolves #433

@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

…pc#433)

Allow custom options as a means of passing data per call to client interceptors
@TLadd TLadd force-pushed the trl/call-options-custom-options-ts branch from 1df7809 to d7da553 Compare July 13, 2018 15:15
@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

@TLadd
Copy link
Contributor Author

TLadd commented Jul 13, 2018

I did sign the CLA. Not sure if something needs to be done to get it to refresh.

@nicolasnoble
Copy link
Member

@TLadd CLA is fine, don't worry.

@murgatroid99 murgatroid99 merged commit b608290 into grpc:master Jul 17, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Oct 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants