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

Add sentiment analysis sample #533

Merged
merged 7 commits into from
Sep 19, 2016
Merged

Add sentiment analysis sample #533

merged 7 commits into from
Sep 19, 2016

Conversation

jerjou
Copy link
Contributor

@jerjou jerjou commented Sep 17, 2016

This PR is an attempt to merge #524, which refuses to allow me to squash+merge because "At least one approved review is required" even though I approved it.

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Sep 17, 2016
@theacodes
Copy link
Contributor

I've approved, but please placate travis before merging.

@jerjou
Copy link
Contributor Author

jerjou commented Sep 19, 2016

@dpebot would you please merge this pull request in the event that Travis approves? You have my gratitude.

@dpebot
Copy link
Collaborator

dpebot commented Sep 19, 2016

Okay! I'll merge when all statuses are green.

@dpebot dpebot added the automerge Merge the pull request once unit tests and other checks pass. label Sep 19, 2016
@theacodes
Copy link
Contributor

@jerjou you're really enjoying your changes to dpebot, aren't you? :D

@jerjou
Copy link
Contributor Author

jerjou commented Sep 19, 2016

Gotta show the robots respect from the start, to secure the upper-class servitude positions when they take over.

@theacodes theacodes merged commit e8a10bf into master Sep 19, 2016
@theacodes theacodes deleted the manshreck branch September 19, 2016 19:41
@jerjou jerjou removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 19, 2016
hkdevandla pushed a commit to hkdevandla/python-language that referenced this pull request Sep 26, 2020
…s-samples#533)

* Add sentiment analysis sample
* Move sample review files into resources directory
* Remove blank line from end of file
* Update set up instructions to point to getting started guide
* Update README to remove need to set up gcloud. Itemize what setting
up a project entails.
* Update NL README to link to Sentiment tutorial code
* Coerce number types before comparison
busunkim96 pushed a commit to googleapis/python-language that referenced this pull request Sep 29, 2020
…s-samples#533)

* Add sentiment analysis sample
* Move sample review files into resources directory
* Remove blank line from end of file
* Update set up instructions to point to getting started guide
* Update README to remove need to set up gcloud. Itemize what setting
up a project entails.
* Update NL README to link to Sentiment tutorial code
* Coerce number types before comparison
m-strzelczyk pushed a commit that referenced this pull request Nov 18, 2022
* Add sentiment analysis sample
* Move sample review files into resources directory
* Remove blank line from end of file
* Update set up instructions to point to getting started guide
* Update README to remove need to set up gcloud. Itemize what setting
up a project entails.
* Update NL README to link to Sentiment tutorial code
* Coerce number types before comparison
parthea pushed a commit to googleapis/google-cloud-python that referenced this pull request Jul 6, 2023
…s-samples#533)

* Add sentiment analysis sample
* Move sample review files into resources directory
* Remove blank line from end of file
* Update set up instructions to point to getting started guide
* Update README to remove need to set up gcloud. Itemize what setting
up a project entails.
* Update NL README to link to Sentiment tutorial code
* Coerce number types before comparison
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: no This human has *not* signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants