Skip to content

chore(librarian): onboard google-ads-admanager#14463

Merged
parthea merged 1 commit intomainfrom
onboard-google-ads-admanager
Sep 18, 2025
Merged

chore(librarian): onboard google-ads-admanager#14463
parthea merged 1 commit intomainfrom
onboard-google-ads-admanager

Conversation

@parthea
Copy link
Copy Markdown
Contributor

@parthea parthea commented Sep 18, 2025

Onboard google-ads-admanager to librarian

Towards googleapis/librarian#761

@parthea parthea marked this pull request as ready for review September 18, 2025 09:52
@parthea parthea requested a review from a team September 18, 2025 09:52
@parthea parthea requested a review from a team as a code owner September 18, 2025 09:52
@parthea parthea requested review from jskeet and suztomo September 18, 2025 09:52
- path: google/ads/admanager/v1
source_roots:
- packages/google-ads-admanager
preserve_regex:
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.

In Go, I've made all our regexes anchored to the source directories to avoid any oddities - see https://github.com/googleapis/google-cloud-go/blob/main/.librarian/state.yaml for example.

No need to change this now, but we might want to align on this (and make sure the behavior is really carefully documented). For Go, this fixed issues where there are modules of both "ai" and "aiplatform".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in #14464

@parthea parthea merged commit 7926fdd into main Sep 18, 2025
23 checks passed
@parthea parthea deleted the onboard-google-ads-admanager branch September 18, 2025 10:24
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