Skip to content

Commit

Permalink
Merge pull request #6439 from planetf1/slf
Browse files Browse the repository at this point in the history
#6143 fix SLF4J dups in maven tests
  • Loading branch information
planetf1 authored Apr 29, 2022
2 parents 2f98f1c + 2718e46 commit d649a0e
Show file tree
Hide file tree
Showing 4 changed files with 26 additions and 11 deletions.
10 changes: 10 additions & 0 deletions open-metadata-implementation/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -45,4 +45,14 @@
<module>user-interfaces</module>
</modules>

<dependencies>
<!-- Use slf4f-simple anywhere as logging implementation in TEST scope only -->
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4j.version}</version>
<scope>test</scope>
</dependency>
</dependencies>

</project>
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,12 @@
</dependency>


<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<scope>test</scope>
</dependency>

</dependencies>

</project>
10 changes: 10 additions & 0 deletions open-metadata-test/open-metadata-fvt/open-types-fvt/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,14 @@
<module>open-types-test</module>
</modules>

<dependencies>
<!-- Use slf4f-simple anywhere as logging implementation in TEST scope only -->
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4j.version}</version>
<scope>test</scope>
</dependency>
</dependencies>

</project>
11 changes: 0 additions & 11 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3360,17 +3360,6 @@
</repository>
</distributionManagement>

<!-- Most dependencies pushed down to keep components minimal -->
<dependencies>
<!-- Use slf4f-simple anywhere as logging implementation in TEST scope only -->
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4j.version}</version>
<scope>test</scope>
</dependency>
</dependencies>

<build>

<pluginManagement>
Expand Down

0 comments on commit d649a0e

Please sign in to comment.