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

Split analyze.js into two separate samples #360

Merged
merged 1 commit into from
Apr 25, 2017
Merged

Split analyze.js into two separate samples #360

merged 1 commit into from
Apr 25, 2017

Conversation

ace-n
Copy link
Contributor

@ace-n ace-n commented Apr 25, 2017

Per @sgreenberg's instructions, we want to have separate samples for v1beta1 and v1beta2 of the NL API.

@ace-n ace-n requested review from jmdobry and sgreenberg April 25, 2017 18:02
@codecov-io
Copy link

Codecov Report

Merging #360 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #360   +/-   ##
=======================================
  Coverage   83.88%   83.88%           
=======================================
  Files           4        4           
  Lines         422      422           
=======================================
  Hits          354      354           
  Misses         68       68

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce94259...5ce6c12. Read the comment docs.

Copy link
Member

@jmdobry jmdobry left a comment

Choose a reason for hiding this comment

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

Are there any actual differences between the v1beta1 and v1beta2 samples?

@ace-n
Copy link
Contributor Author

ace-n commented Apr 25, 2017

Two main ones:

  • v1beta2 supports more languages than v1beta1 (i.e. German and a few others). This is covered in the tests
  • v1beta2 introduces the analyzeEntitySentiment method.

@ace-n ace-n merged commit a8e7c19 into master Apr 25, 2017
@ace-n ace-n deleted the split-nl branch April 25, 2017 19:58
ahrarmonsur pushed a commit that referenced this pull request Nov 8, 2022
* feat: monitoring conversion by delete js file and add ts from generator

* upgrade package.json

* fixed samples test by modify the matched path

* prelint, add eslint for samples test

* fix upstream proto links, get rid of synth hack

Co-authored-by: Benjamin E. Coe <bencoe@google.com>
telpirion pushed a commit that referenced this pull request Nov 9, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
telpirion pushed a commit that referenced this pull request Nov 9, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
ace-n pushed a commit that referenced this pull request Nov 10, 2022
* feat: monitoring conversion by delete js file and add ts from generator

* upgrade package.json

* fixed samples test by modify the matched path

* prelint, add eslint for samples test

* fix upstream proto links, get rid of synth hack

Co-authored-by: Benjamin E. Coe <bencoe@google.com>
kweinmeister pushed a commit that referenced this pull request Nov 10, 2022
* updated CHANGELOG.md

* updated package.json

* updated samples/package.json
NimJay pushed a commit that referenced this pull request Nov 10, 2022
NimJay pushed a commit that referenced this pull request Nov 10, 2022
ace-n pushed a commit that referenced this pull request Nov 11, 2022
* updated CHANGELOG.md

* updated package.json

* updated samples/package.json
ace-n pushed a commit that referenced this pull request Nov 11, 2022
* updated CHANGELOG.md

* updated package.json

* updated samples/package.json
ahrarmonsur pushed a commit that referenced this pull request Nov 16, 2022
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/cc99acfa-05b8-434b-9500-2f6faf2eaa02/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@799d8e6
ace-n pushed a commit that referenced this pull request Nov 17, 2022
ace-n pushed a commit that referenced this pull request Nov 17, 2022
ace-n pushed a commit that referenced this pull request Nov 17, 2022
ace-n pushed a commit that referenced this pull request Nov 17, 2022
ace-n pushed a commit that referenced this pull request Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants