Skip to content

refactor(auth): Remove unused dependencies region tag - #10342

Open
Kef131 wants to merge 1 commit into
mainfrom
refactor(auth)_remove_unused_dependencies_regiontag
Open

refactor(auth): Remove unused dependencies region tag#10342
Kef131 wants to merge 1 commit into
mainfrom
refactor(auth)_remove_unused_dependencies_regiontag

Conversation

@Kef131

@Kef131 Kef131 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes #
b/347826218

Removed dependencies with no reference on samples.

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

@product-auto-label product-auto-label Bot added samples Issues that are directly related to samples. api: auth labels Aug 27, 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 cleans up the auth/pom.xml file by removing redundant dependency marker comments (<!-- START dependencies --> and <!-- END dependencies -->). No review comments were provided, and there is no additional feedback to address.

@angelcaamal angelcaamal 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!

@Kef131
Kef131 marked this pull request as ready for review August 28, 2026 17:02
@Kef131
Kef131 requested review from a team and yoshi-approver as code owners August 28, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: auth samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants