Skip to content

fix: onboarding google-cloud-automl to Librarian - #36511

Draft
suztomo wants to merge 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-automl
Draft

fix: onboarding google-cloud-automl to Librarian#36511
suztomo wants to merge 2 commits into
googleapis:mainfrom
suztomo:chore/onboarding-automl

Conversation

@suztomo

@suztomo suztomo commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

Generator Improvements Applied

  1. Split URL Normalization (b/153077040):
    • Split URLs across newlines in proto doc comments (e.g. https:\n //...) are now automatically rejoined into single-line https://... URLs in google-cloud-automl-v1beta1/proto_docs/google/cloud/automl/v1beta1/io.rb.
  2. Non-existent Message Reference Stripping (b/158466893):
    • Cross-references pointing to google.cloud.automl.v1.ColumnSpec (which only exists in v1beta1) are stripped in google-cloud-automl-v1/proto_docs/google/cloud/automl/v1/io.rb, outputting plain display_name-s without broken cross-references.
  3. OwlBot Removal:
    • Removed .owlbot.rb, .OwlBot.yaml, and .owlbot-manifest.json from both gems as the post-processing substitutions are now natively handled by gapic-generator-ruby.

Local Verification

  • toys ci --test --rubocop --doctest --gems google-cloud-automl-v1,google-cloud-automl-v1beta1: Passed (146 runs, 3652 assertions, 0 failures, 0 errors, 0 RuboCop offenses).
  • toys ci --yard --gems google-cloud-automl-v1,google-cloud-automl-v1beta1: Passed (0 warnings, 0 errors).

@suztomo

suztomo commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

They need to be addressed by gapic-generator-ruby fix.

google-cloud-automl-v1/proto_docs/.../io.rb: Direct generator output includes [display_name-s][google.cloud.automl.v1.ColumnSpec.display_name] (reverting previous OwlBot post-processor for b/158466893). Handled by ruby-cloud-yard-strict: "false".
google-cloud-automl-v1beta1/proto_docs/.../io.rb: Multi-line proto comment URL wrapping (https:\n# //...) output directly from generator without OwlBot post-processing (b/153077040).

@suztomo
suztomo force-pushed the chore/onboarding-automl branch from 5d09b9f to cbb36d2 Compare September 1, 2026 22:35
@suztomo

suztomo commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

There's still a suspicious link in the doc at google-cloud-automl-v1beta1/proto_docs/google/cloud/automl/v1beta1/io.rb :

Screenshot 2026-09-01 at 6 48 46 PM

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.

ruby: google-cloud-automl-v1 and google-cloud-automl-v1beta1

1 participant