Skip to content

fix(healthcare): remove old healthcare java dependency region tag PR Step 2/2 #10282

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Kef131 wants to merge 2 commits into from
Closed

fix(healthcare): remove old healthcare java dependency region tag PR Step 2/2 #10282

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0dbf462
chore: change regiontag healthcare dependencies in pom.xml
Kef131 Jun 15, 2026
78ba41d
refactor: remove old region tag dependency markers from healthcare sa…
Kef131 Jun 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 9 additions & 12 deletions healthcare/v1/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,9 +53,7 @@
<maven.compiler.source>11</maven.compiler.source>
</properties>

<!-- [START dependencies] -->
<!-- Using libraries-bom to manage versions.
See https://github.com/GoogleCloudPlatform/cloud-opensource-java/wiki/The-Google-Cloud-Platform-Libraries-BOM -->
<!-- [START healthcare_java_dependencies] -->
<dependencyManagement>
<dependencies>
<dependency>
Expand All @@ -69,19 +67,20 @@
</dependencyManagement>

<dependencies>
<dependency>
<groupId>com.google.http-client</groupId>
<artifactId>google-http-client-jackson2</artifactId>
</dependency>
<dependency>
<groupId>com.google.apis</groupId>
<artifactId>google-api-services-healthcare</artifactId>
<version>v1-rev20240130-2.0.0</version>
</dependency>
<!-- [END healthcare_java_dependencies] -->
<dependency>
<groupId>com.google.http-client</groupId>
<artifactId>google-http-client-jackson2</artifactId>
</dependency>
<dependency>
<groupId>com.google.api-client</groupId>
<artifactId>google-api-client</artifactId>
</dependency>
</dependency>
<dependency>
<groupId>com.google.auth</groupId>
<artifactId>google-auth-library-oauth2-http</artifactId>
Expand All @@ -91,8 +90,6 @@
<artifactId>google-cloud-core</artifactId>
<scope>compile</scope>
</dependency>
<!-- [END dependencies] -->

<!-- Test dependencies -->
<dependency>
<groupId>com.google.truth</groupId>
Expand All @@ -111,7 +108,7 @@
<artifactId>httpmime</artifactId>
<version>4.5.14</version>
</dependency>
<!-- [START dependencies] -->
<!-- [START healthcare_java_dependencies] -->
</dependencies>
<!-- [END dependencies] -->
<!-- [END healthcare_java_dependencies] -->
Comment on lines +111 to +113

@gemini-code-assist gemini-code-assist Bot Jun 17, 2026

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.

medium

This second pair of healthcare_java_dependencies region tags is redundant and creates a duplicate region tag definition in the same file. Duplicate region tags can cause parsing errors or incorrect snippet generation in the documentation pipeline. Since the first block (lines 56-75) already defines this region, this second pair should be removed.

  </dependencies>

</project>