Skip to content

fix: onboarding google-cloud-app_optimize to Librarian - #36509

Merged
suztomo merged 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-app-optimize
Sep 1, 2026
Merged

fix: onboarding google-cloud-app_optimize to Librarian#36509
suztomo merged 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-app-optimize

Conversation

@suztomo

@suztomo suztomo commented Sep 1, 2026

Copy link
Copy Markdown
Member

Onboarding google-cloud-app_optimize and google-cloud-app_optimize-v1beta to Librarian

Tracking issue: googleapis/librarian#7464
Tracker: go/sdk-librarian-ruby-tracker

Summary of Libraries Onboarded

  • google-cloud-app_optimize (wrapper gem)
  • google-cloud-app_optimize-v1beta (versioned GAPIC client)

Changes Overview

  1. OwlBot Removal: Removed .OwlBot.yaml and .owlbot-manifest.json from both google-cloud-app_optimize and google-cloud-app_optimize-v1beta.
  2. README Summary Deduplication: Removed duplicated summary sentence in README.md for both gems caused by legacy generator/template output.

Context & Generator Configuration

  • In googleapis/googleapis, the service proto path is google/cloud/appoptimize/v1beta.
  • In google/cloud/appoptimize/v1beta/BUILD.bazel and google/cloud/appoptimize/BUILD.bazel, the Ruby GAPIC rules specified ruby-cloud-gem-name=google-cloud-app_optimize-v1beta and ruby-cloud-gem-name=google-cloud-app_optimize.
  • Under Librarian, removing skip_generate: true in librarian.yaml generates and updates the gems cleanly using the configured library names.

Verification

  • toys ci --test --rubocop --doctest --gems google-cloud-app_optimize,google-cloud-app_optimize-v1beta:
    • google-cloud-app_optimize-v1beta: 24 runs, 518 assertions, 0 failures, 0 errors. RuboCop: 21 files inspected, 0 offenses detected.
    • google-cloud-app_optimize: 3 runs, 1 assertion, 0 failures, 0 errors, 2 skips. RuboCop: 4 files inspected, 0 offenses detected.
  • bundle exec yard doc --fail-on-warning: Passed with 0 errors and 0 warnings for both gems.

@suztomo
suztomo merged commit 8686263 into googleapis:main Sep 1, 2026
19 checks passed
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