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

Natural Language GAPIC client library #1018

Merged
merged 16 commits into from
Jul 28, 2017

Conversation

dizcology
Copy link
Member

@dizcology dizcology commented Jul 12, 2017

migrate samples to natural language gapic client library

Merge only after the client library has been released and tested.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 12, 2017
@dizcology dizcology changed the title migrate samples to natural language gapic client library Natural Language GAPIC client library Jul 12, 2017

encoding = enums.EncodingType.UTF32
encoding = 'UTF32'
Copy link
Contributor

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?

Copy link
Member Author

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

Copy link
Member Author

@dizcology dizcology Jul 28, 2017

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.

@theacodes
Copy link
Contributor

This LGTM, but you need to resolve the conflicts first.

@dizcology
Copy link
Member Author

client library version 0.26.1 released, and tested against the updated samples.

@theacodes theacodes merged commit 7c3bebd into GoogleCloudPlatform:master Jul 28, 2017
busunkim96 pushed a commit to busunkim96/python-language that referenced this pull request May 20, 2020
hkdevandla pushed a commit to hkdevandla/python-language that referenced this pull request Sep 26, 2020
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants