FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Language: Add `classify_text` method (v1). by lukesneeringer · Pull Request #4283 · googleapis/google-cloud-python · GitHub

Language: Add classify_text method (v1). - #4283

Merged
lukesneeringer merged 5 commits into
masterfrom
language-v1-updates
Nov 2, 2017
Merged

Language: Add classify_text method (v1).#4283
lukesneeringer merged 5 commits into
masterfrom
language-v1-updates

Conversation

Copy link
Copy Markdown
Contributor

This PR adds the classify_text method currently in the v1beta2 API version to the v1 version. This should be held until November 15 (the endpoint will not be live on v1 until then).

lukesneeringer added api: language Issues related to the Cloud Natural Language API API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Oct 30, 2017
lukesneeringer self-assigned this Oct 30, 2017
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 30, 2017

dhermes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Cursory LGTM

lukesneeringer removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 1, 2017

Copy link
Copy Markdown
Contributor Author

Okay, I tested this and am happy with it (but need to see what is up with CI).

@dhermes, I want to do the following:

  • I want to launch a new beta with these changes imminently (today if possible).
  • Next week (probably), I want to promote this to GA.

How does this interact with the releases you just did? I want to make sure I do not step on your toes.

dhermes commented Nov 1, 2017

Copy link
Copy Markdown
Contributor

@lukesneeringer I think it'd be fine to do two releases only (since you don't depend on a new version of core):

  • google-cloud-language==0.31.0
  • google-cloud==0.29.0 (which will have as the only change the dependence on the 0.31.x series of language

dhermes force-pushed the language-v1-updates branch from 0155102 to fa1d8cd Compare November 1, 2017 23:35

Copy link
Copy Markdown
Contributor Author

@dhermes Sounds good.

lukesneeringer merged commit 9247b99 into master Nov 2, 2017
lukesneeringer deleted the language-v1-updates branch November 2, 2017 15:24
parthea pushed a commit that referenced this pull request Jul 6, 2023
* Language: Add `classify_text` method (v1).
* Update v1beta2 also.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: language Issues related to the Cloud Natural Language API API. 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


Back | FazBrowse Home | New Git URL