Skip to content
This repository has been archived by the owner on Sep 9, 2023. It is now read-only.

Commit

Permalink
fix: declaring test-scope artifact as runtime (#1014)
Browse files Browse the repository at this point in the history
Within this repository this dependency is only used in tests.
However, making this test scope causes undeclared dependencies
after the flatten-maven-plugin.

Similar to googleapis/java-pubsub#1239
  • Loading branch information
suztomo committed Aug 8, 2022
1 parent b170504 commit f90cc12
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions google-cloud-aiplatform/pom.xml
Expand Up @@ -78,6 +78,17 @@
<groupId>com.google.api.grpc</groupId>
<artifactId>grpc-google-iam-v1</artifactId>
</dependency>
<dependency>
<groupId>com.google.code.gson</groupId>
<artifactId>gson</artifactId>
<!--
Within this repository this dependency is only used in tests. However,
making this test scope causes undeclared dependencies after the
flatten-maven-plugin.
https://github.com/mojohaus/flatten-maven-plugin/issues/185
-->
<scope>runtime</scope>
</dependency>

<!-- Test dependencies -->
<dependency>
Expand Down Expand Up @@ -108,11 +119,6 @@
<groupId>com.google.protobuf</groupId>
<artifactId>protobuf-java-util</artifactId>
</dependency>
<dependency>
<groupId>com.google.code.gson</groupId>
<artifactId>gson</artifactId>
<scope>test</scope>
</dependency>
</dependencies>

<profiles>
Expand Down

0 comments on commit f90cc12

Please sign in to comment.