Skip to content

Commit

Permalink
Avoid having an extension dependency in quarkus-grpc-api
Browse files Browse the repository at this point in the history
It is not an extension so it should not depend on extensions.
  • Loading branch information
gsmet committed Aug 6, 2024
1 parent 90e86df commit 484088f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 16 deletions.
7 changes: 4 additions & 3 deletions extensions/grpc/api/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,10 @@
<name>Quarkus - gRPC - API</name>

<dependencies>
<!-- Do not depend on the ArC extension here as we are not in an extension -->
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-arc</artifactId>
<groupId>io.quarkus.arc</groupId>
<artifactId>arc</artifactId>
</dependency>
<dependency>
<groupId>io.grpc</groupId>
Expand All @@ -39,4 +40,4 @@
</dependency>
</dependencies>

</project>
</project>
13 changes: 0 additions & 13 deletions integration-tests/grpc-cli/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -30,19 +30,6 @@
</dependency>

<!-- Minimal test dependencies to *-deployment artifacts for consistent build order -->
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-smallrye-context-propagation-deployment</artifactId>
<version>${project.version}</version>
<type>pom</type>
<scope>test</scope>
<exclusions>
<exclusion>
<groupId>*</groupId>
<artifactId>*</artifactId>
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-grpc-deployment</artifactId>
Expand Down

0 comments on commit 484088f

Please sign in to comment.