-
Notifications
You must be signed in to change notification settings - Fork 439
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
Add iot #937
Add iot #937
Conversation
Updated with partial veneer, PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small item, otherwise looks great
src/Iot/V1/DeviceManagerClient.php
Outdated
throw new InvalidArgumentException( | ||
'The "grpc" transport is not currently supported, ' . | ||
'please use the "rest" transport.'); | ||
} else { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Updated to new repo structure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should wait to merge this until we have merged #156 and tagged a new version of GAX, and then updated this repository for that version. Otherwise, this is good to go.
looks good again! |
Funny travis failure, that repeated when I tried to re-run the build. Have you guys seen that one before? |
I've been trying to track down a travis-only segfault in #923 as well. I don't recall seeing a persistent segfault only on PHP 7.0 though. |
Same here, that seems new. |
Don't merge, pending googleapis/gax-php#149 and partial veneer to allow REST only