Skip to content

feat(translate-v2): enable self-signed JWT by default#33974

Open
quartzmo wants to merge 1 commit into
googleapis:mainfrom
quartzmo:translate-v2-ssjwt
Open

feat(translate-v2): enable self-signed JWT by default#33974
quartzmo wants to merge 1 commit into
googleapis:mainfrom
quartzmo:translate-v2-ssjwt

Conversation

@quartzmo
Copy link
Copy Markdown
Member

  • Update Gemfile to allow google-style ~> 1.31.1.
  • Add ostruct to Gemfile for Ruby 4.0.0+ compatibility.
  • Fix test stubs to accept positional hash for Ruby 3+ keyword separation.

* Update Gemfile to allow google-style ~> 1.31.1.
* Add ostruct to Gemfile for Ruby 4.0.0+ compatibility.
* Fix test stubs to accept positional hash for Ruby 3+ keyword separation.
@quartzmo quartzmo requested review from a team and yoshi-approver as code owners May 20, 2026 19:24
gem "google-cloud-core", path: "../google-cloud-core"
gem "google-cloud-errors", path: "../google-cloud-errors"
gem "google-style", "~> 1.30.1"
gem "google-style", "~> 1.31.1"
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.

1.32 please

gem "google-cloud-errors", path: "../google-cloud-errors"
gem "google-style", "~> 1.30.1"
gem "google-style", "~> 1.31.1"
gem "ostruct"
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.

Nit: add a version, like ~> 0.6.3

# translation.text #=> "Salve mundi!"
#
def self.new project_id: nil, credentials: nil, key: nil, scope: nil, retries: nil, timeout: nil, endpoint: nil
def self.new project_id: nil, credentials: nil, key: nil, scope: nil, retries: nil, timeout: nil, endpoint: nil, enable_self_signed_jwt: nil
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.

Missing docstring for enable_self_signed_jwt

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.

2 participants