Skip to content

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