fix: Add missing used dependency and remove force-usage for dependenc…

…y-plugin for a now explicitly used dependency (#7364)
jeremylong · Jan 29, 2025 · 64089f3 · 64089f3
2 parents ad568cc + 3112504
commit 64089f3
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/core/pom.xml b/core/pom.xml
@@ -228,6 +228,11 @@ Copyright (c) 2012 Jeremy Long. All Rights Reserved.
             <artifactId>logback-classic</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>ch.qos.logback</groupId>
+            <artifactId>logback-core</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.owasp</groupId>
             <artifactId>dependency-check-utils</artifactId>
@@ -506,9 +511,6 @@ Copyright (c) 2012 Jeremy Long. All Rights Reserved.
                             <version>${maven-dependency-plugin.version}</version>
                             <configuration>
                                 <usedDependencies combine.children="append">
-                                    <!-- logback is our logging implementation during test and is test-scoped due to a lack of a
-                                    test-runtime scope - it should be considered 'used' in the context of dependency:analyze-report -->
-                                    <usedDependency>ch.qos.logback:logback-classic</usedDependency>
                                     <!-- dependencies to be copied for use in unit/integration testcases are, due to
                                     lack of a test-runtime scope, configured as test-scoped / optional and should be
                                     considered used for dependency:analyze-report -->