Skip to content

Librarian generate pull request: 20250911T193301Z#14418

Closed
cloud-sdk-librarian-robot wants to merge 1 commit intomainfrom
librarian-20250911T193301Z
Closed

Librarian generate pull request: 20250911T193301Z#14418
cloud-sdk-librarian-robot wants to merge 1 commit intomainfrom
librarian-20250911T193301Z

Conversation

@cloud-sdk-librarian-robot
Copy link
Copy Markdown
Collaborator

This pull request is generated with proto changes between
googleapis/googleapis@d300b15
(exclusive) and
googleapis/googleapis@2585041
(inclusive).

Librarian Version: not available
Language Image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator:latest

BEGIN_COMMIT_OVERRIDE

BEGIN_NESTED_COMMIT
feat: [google-cloud-dlp] add LocationSupport,Domain,DocumentFallbackLocation
docs: minor doc revision

PiperOrigin-RevId: 804670680

Source-link: googleapis/googleapis@2585041
END_NESTED_COMMIT

END_COMMIT_OVERRIDE

Copy link
Copy Markdown
Contributor

@parthea parthea left a comment

Choose a reason for hiding this comment

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

@@ -1,23 +1,24 @@
image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator:latest
libraries:
- id: google-cloud-dlp
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.

Hmm our indentation isn't correct for the previous state.yaml. @parthea do we want to update the script that is used to create /update this state.yaml?

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.

Can we consider this non-blocking and file a bug to follow up?

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.

I filed googleapis/librarian#2073 to follow up on this

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 formatting was fixed in #14419 by manually running yamlfmt. We'll close this PR and open a new one as there are conflicts in this PR

@parthea parthea closed this Sep 11, 2025
@parthea parthea reopened this Sep 11, 2025
@parthea parthea closed this Sep 11, 2025
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.

4 participants