Skip to content

fix: onboarding google-cloud-privileged_access_manager to Librarian - #36485

Merged
suztomo merged 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-privileged-access-manager
Aug 31, 2026
Merged

fix: onboarding google-cloud-privileged_access_manager to Librarian#36485
suztomo merged 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-privileged-access-manager

Conversation

@suztomo

@suztomo suztomo commented Aug 31, 2026

Copy link
Copy Markdown
Member

Onboarding google-cloud-privileged_access_manager to Librarian

Summary of Libraries Onboarded

  • google-cloud-privileged_access_manager (wrapper)
  • google-cloud-privileged_access_manager-v1 (versioned client)

Changes Overview

  1. OwlBot Removal: Removed .OwlBot.yaml, .owlbot-manifest.json, and .owlbot.rb from both gems.
  2. Library Metadata / Description Normalization:
    • In legacy Bazel/OwlBot builds, the generator rendered the full documentation.overview markdown section from privilegedaccessmanager_v1.yaml into README.md and gemspec description, which contained unescaped {projects|folders|organizations} patterns requiring an .owlbot.rb regex post-processor to prevent YARD parsing errors (Properly escape README.md gapic-generator-ruby#1094).
    • Under Librarian, ruby-cloud-description and ruby-cloud-summary are both populated from documentation.summary, omitting the overview block and cleanly standardizing README.md and .gemspec descriptions across wrapper and versioned client gems.
    • For detailed analysis, see ruby: escaping links in readme librarian#7446.

Verification

  • toys ci --test --rubocop --doctest --gems google-cloud-privileged_access_manager,google-cloud-privileged_access_manager-v1: Passed (43 runs, 1116 assertions, 0 offenses).
  • bundle exec yard doc --fail-on-warning: Passed with 0 errors and 0 warnings.

@suztomo
suztomo marked this pull request as ready for review August 31, 2026 15:45
@suztomo
suztomo requested review from a team and yoshi-approver as code owners August 31, 2026 15:45
@suztomo
suztomo merged commit 1dac6e2 into googleapis:main Aug 31, 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