Skip to content

chore: modernize post-processing for 10 libraries#13815

Merged
blakeli0 merged 1 commit into
googleapis:mainfrom
yangyzs:chore-modernize-owlbot-batch-1
Jul 18, 2026
Merged

chore: modernize post-processing for 10 libraries#13815
blakeli0 merged 1 commit into
googleapis:mainfrom
yangyzs:chore-modernize-owlbot-batch-1

Conversation

@yangyzs

@yangyzs yangyzs commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Remove legacy owlbot.py scripts and migrate configurations to librarian.yaml for the following libraries:

  • bigquerydatatransfer
  • bigquerystorage
  • bigtable
  • containeranalysis
  • datastore
  • dialogflow
  • dlp
  • errorreporting
  • firestore
  • grafeas

For googleapis/librarian#6900

Remove legacy owlbot.py scripts and migrate configurations to librarian.yaml
for the following libraries:
- bigquerydatatransfer
- bigquerystorage
- bigtable
- containeranalysis
- datastore
- dialogflow
- dlp
- errorreporting
- firestore
- grafeas

For googleapis/librarian#6900
@yangyzs
yangyzs requested review from a team as code owners July 17, 2026 20:15

@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 migrates post-processing steps from legacy Python synthesis scripts to the librarian.yaml configuration across multiple Java libraries, alongside updating README files with Gradle dependencies and billing prerequisites. The code review highlights several critical omissions and errors in the newly added librarian.yaml configurations, including a regex pattern in java-bigtable missing multiline flags, a missing test file replacement in java-dialogflow, a missing import statement addition in java-errorreporting, and missing OAuth scope removals in java-grafeas.

Comment thread librarian.yaml
Comment thread librarian.yaml
Comment thread librarian.yaml
Comment thread librarian.yaml
@yangyzs
yangyzs requested review from JoeWang1127 and blakeli0 July 17, 2026 20:40
Comment thread java-firestore/README.md

@blakeli0 blakeli0 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.

The Java changes LGTM. Deferring to @JoeWang1127 regarding the librarian.yaml changes.

@JoeWang1127 JoeWang1127 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.

librarian.yaml looks fine.

@blakeli0
blakeli0 merged commit d5e705b into googleapis:main Jul 18, 2026
386 of 391 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.

3 participants