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

PHP: Fix property reference in BaseStub::close() #3858

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

enisoc
Copy link
Contributor

@enisoc enisoc commented Oct 15, 2015

No description provided.

@grpc-kokoro
Copy link

Can one of the admins verify this patch?

1 similar comment
@grpc-kokoro
Copy link

Can one of the admins verify this patch?

@enisoc
Copy link
Contributor Author

enisoc commented Oct 20, 2015

@jtattermusch @murgatroid99
Pinging some reviewers of previous PHP-related pull requests. Could someone take a look at this? Thanks.

@jtattermusch
Copy link
Contributor

ok to test this

@stanley-cheung
Copy link
Contributor

Thanks for the contribution!

I have almost the exact same change locally to improve on our unit tests and code coverage analysis :) This looks good to merge. LGTM.

@stanley-cheung
Copy link
Contributor

Waiting for Jenkins to finish.

stanley-cheung added a commit that referenced this pull request Oct 20, 2015
PHP: Fix property reference in BaseStub::close()
@stanley-cheung stanley-cheung merged commit 42e2beb into grpc:master Oct 20, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants