Skip to content

fix(healthcare): rename healthcare java dependency region tag PR Step 1/2#10281

Open
Kef131 wants to merge 2 commits into
mainfrom
fix(healthcare)-rename_reuse_healthcare_region_tag
Open

fix(healthcare): rename healthcare java dependency region tag PR Step 1/2#10281
Kef131 wants to merge 2 commits into
mainfrom
fix(healthcare)-rename_reuse_healthcare_region_tag

Conversation

@Kef131

@Kef131 Kef131 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes b/383619440

Rename region tag dependencies to healthcare_dependencies in healthcare/v1/pom.xml.

Checklist

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • These samples need a new API enabled in testing projects to pass (let us know which ones)
  • These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • Tests pass: mvn clean verify required (Requires running gcert first to authenticate with Airlock to resolve dependencies)
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
  • 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
  • Please merge this PR for me once it is approved

Checklist

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • These samples need a new API enabled in testing projects to pass (let us know which ones)
  • These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • 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
  • 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
  • Please merge this PR for me once it is approved

@product-auto-label product-auto-label Bot added api: healthcare Issues related to the Cloud Healthcare API API. samples Issues that are directly related to samples. api: healhcare labels Jun 15, 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 updates the healthcare/v1/pom.xml file by introducing new region tags (healthcare_java_dependencies) and reorganizing the dependencies. Specifically, the google-api-services-healthcare dependency is now enclosed within the region tags, while other dependencies like google-http-client-jackson2 and google-api-client have been moved outside of them. Some minor indentation issues were also corrected. I have no feedback to provide.

@Kef131 Kef131 marked this pull request as ready for review June 16, 2026 00:28
@Kef131 Kef131 requested review from a team and yoshi-approver as code owners June 16, 2026 00:28
@snippet-bot

snippet-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 2 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: healthcare Issues related to the Cloud Healthcare API API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants