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

Update dependency @google-cloud/translate to v5 - autoclosed #1531

Conversation

renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Oct 28, 2019

This PR contains the following updates:

Package Type Update Change
@google-cloud/translate dependencies major ^4.0.0 -> ^5.0.0

Release Notes

googleapis/nodejs-translate

v5.0.1

Compare Source

v5.0.0

Compare Source

⚠ BREAKING CHANGES
  • this significantly changes TypeScript types and API surface from the v2 API. Reference samples/ for help making the migration from v2 to v3.
Features
Bug Fixes
4.2.1 (2019-10-22)
Bug Fixes

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot. View repository job log here.

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 28, 2019
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 28, 2019
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 28, 2019
@fhinkel fhinkel added the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@fhinkel
Copy link
Contributor

fhinkel commented Oct 29, 2019

👋 @ricalo Could you have a look at the failing speech to speech test please.

@fhinkel fhinkel removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@ricalo
Copy link
Member

ricalo commented Oct 29, 2019

@ricalo Could you have a look at the failing speech to speech test please.

It looks like the test could not find the @google-cloud/translate dependency updated in this PR. I can take a closer look later today. Not sure if it's just a temporary issue, but I'd run the tests again in the meantime.

Edit: The ocr-app fails with the same error.

@ricalo ricalo added the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@ricalo
Copy link
Member

ricalo commented Oct 29, 2019

I'd definitely run the tests again since the google-cloud/translate team released a fix for a bad publish of the library. Check googleapis/nodejs-translate/pull/361.

Note that google-cloud/translate 5.0.0 also introduced some breaking changes. I'm not sure if that's affecting this PR.

The kokoro:run label doesn't work for me. It looks like one of the repo owners must trigger a rebuild. @fhinkel could you help with this?

@fhinkel fhinkel added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@fhinkel
Copy link
Contributor

fhinkel commented Oct 29, 2019

Looks to me like Kokoro was run again after you applied the label. Running it one more time just in case.

@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 29, 2019
@grant grant removed their request for review October 29, 2019 16:45
@fhinkel fhinkel assigned ace-n and unassigned fhinkel and ricalo Oct 29, 2019
@ricalo
Copy link
Member

ricalo commented Oct 30, 2019

To update to version ^5.0.0, the speech-to-speech sample (and I assume the ocr-app sample too) should update the line that creates a new Translate object from:

const {Translate} = require('@google-cloud/translate');

to

const {Translate} = require('@google-cloud/translate').v2;

The sample builds fine locally with this change. @ace-n, let me know if you agree and I'll get a PR ready.

@ace-n
Copy link
Contributor

ace-n commented Oct 30, 2019 via email

@renovate-bot renovate-bot changed the title Update dependency @google-cloud/translate to v5 Update dependency @google-cloud/translate to v5 - autoclosed Oct 30, 2019
@renovate-bot renovate-bot deleted the renovate/google-cloud-translate-5.x branch October 30, 2019 18:41
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.

6 participants