Skip to content

fix(security-center): Added missed region tags to Finding/Source snipepts files - #10327

Open
Kef131 wants to merge 2 commits into
mainfrom
fix(security-center)_add_missed_regiontags
Open

fix(security-center): Added missed region tags to Finding/Source snipepts files#10327
Kef131 wants to merge 2 commits into
mainfrom
fix(security-center)_add_missed_regiontags

Conversation

@Kef131

@Kef131 Kef131 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes #
b/527130933

Adds missed region tags to the Security Command Center from previous migrations

Checklist

Testing

  • I have tested this change on a live environment and verified it works as intended.
  • Tests pass: mvn clean verify required
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only

Compliance & Style

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample

Post-Approval Actions

  • Please merge this PR for me once it is approved

@Kef131 Kef131 self-assigned this Aug 11, 2026
@product-auto-label product-auto-label Bot added api: securitycenter Issues related to the Security Command Center API. samples Issues that are directly related to samples. labels Aug 11, 2026

@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 adds region tags to define documentation snippets in FindingSnippets.java and SourceSnippets.java. The feedback recommends adding a newline character at the end of SourceSnippets.java to comply with the Google Java Style Guide.

…uritycenter/SourceSnippets.java

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

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

LGTM

@amcolin
amcolin marked this pull request as ready for review August 11, 2026 21:36
@amcolin
amcolin requested review from a team and yoshi-approver as code owners August 11, 2026 21:36
@snippet-bot

snippet-bot Bot commented Aug 11, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 18 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@amcolin amcolin added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 11, 2026
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: securitycenter Issues related to the Security Command Center API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants