Skip to content

chore: revert "chore: migrate google-cloud-translate to librarian"#8624

Open
feywind wants to merge 1 commit into
mainfrom
feywind-revert-translate
Open

chore: revert "chore: migrate google-cloud-translate to librarian"#8624
feywind wants to merge 1 commit into
mainfrom
feywind-revert-translate

Conversation

@feywind

@feywind feywind commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

This reverts commit f0bd3aa. We don't want to make a major here yet.

This reverts commit f0bd3aa. We don't want to
make a major here yet.
@feywind feywind requested a review from a team as a code owner June 12, 2026 15:40

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces the v2 Translation client along with its unit tests, integrates it into the main entry point, and updates the repository metadata and OwlBot configuration. The review feedback focuses on improving the robustness of the new client by preventing potential runtime crashes. Specifically, it recommends verifying that the input is a string before passing it to isHtml, and using optional chaining when accessing properties on API responses (detections, languages, and translations) to handle malformed or empty responses gracefully.

Comment thread packages/google-cloud-translate/src/v2/index.ts
Comment thread packages/google-cloud-translate/src/v2/index.ts
Comment thread packages/google-cloud-translate/src/v2/index.ts
Comment thread packages/google-cloud-translate/src/v2/index.ts
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.

1 participant