-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Natural Language GAPIC client library #1018
Natural Language GAPIC client library #1018
Conversation
|
||
encoding = enums.EncodingType.UTF32 | ||
encoding = 'UTF32' |
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.
can you not use the enums here (should be types.EncodingType.UTF32
?
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.
removing encoding, since it is no longer needed: googleapis/googleapis#408
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.
turns out we should still keep the encoding (added back)
The client libraries are current designed to have all enums in the enums module, such as google.cloud.language.enums
.
This LGTM, but you need to resolve the conflicts first. |
7c7f2ed
to
4777e50
Compare
client library version 0.26.1 released, and tested against the updated samples. |
migrate samples to natural language gapic client library
Merge only after the client library has been released and tested.