fix: PMD warnings with new version (#936)

* chore(deps): update dependency org.apache.maven.plugins:maven-pmd-plugin to v3.22.0 * removed redundancy from eventdetails, SuppressWarnings in OpenfeatureAPI and Client Signed-off-by: DBlanchard88 <davidblanchard88@gmail.com> * Update src/main/java/dev/openfeature/sdk/OpenFeatureClient.java Signed-off-by: Todd Baert <todd.baert@dynatrace.com> --------- Signed-off-by: DBlanchard88 <davidblanchard88@gmail.com> Signed-off-by: Todd Baert <todd.baert@dynatrace.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Todd Baert <todd.baert@dynatrace.com>
open-feature · May 15, 2024 · 1a46aea · 1a46aea
1 parent 0844c8f
commit 1a46aea
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/pom.xml b/pom.xml
@@ -334,7 +334,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-pmd-plugin</artifactId>
-        <version>3.21.2</version>
+        <version>3.22.0</version>
         <executions>
           <execution>
             <id>run-pmd</id>

diff --git a/src/main/java/dev/openfeature/sdk/EventDetails.java b/src/main/java/dev/openfeature/sdk/EventDetails.java
@@ -13,14 +13,14 @@ public class EventDetails extends ProviderEventDetails {
     private String providerName;
 
     static EventDetails fromProviderEventDetails(ProviderEventDetails providerEventDetails, String providerName) {
-        return EventDetails.fromProviderEventDetails(providerEventDetails, providerName, null);
+        return fromProviderEventDetails(providerEventDetails, providerName, null);
     }
 
     static EventDetails fromProviderEventDetails(
             ProviderEventDetails providerEventDetails,
             String providerName,
             String clientName) {
-        return EventDetails.builder()
+        return builder()
                 .clientName(clientName)
                 .providerName(providerName)
                 .flagsChanged(providerEventDetails.getFlagsChanged())

diff --git a/src/main/java/dev/openfeature/sdk/OpenFeatureAPI.java b/src/main/java/dev/openfeature/sdk/OpenFeatureAPI.java
@@ -18,6 +18,7 @@
  * Configuration here will be shared across all {@link Client}s.
  */
 @Slf4j
+@SuppressWarnings("PMD.UnusedLocalVariable")
 public class OpenFeatureAPI implements EventBus<OpenFeatureAPI> {
     // package-private multi-read/single-write lock
     static AutoCloseableReentrantReadWriteLock lock = new AutoCloseableReentrantReadWriteLock();

diff --git a/src/main/java/dev/openfeature/sdk/OpenFeatureClient.java b/src/main/java/dev/openfeature/sdk/OpenFeatureClient.java
@@ -19,7 +19,8 @@
  * {@inheritDoc}
  */
 @Slf4j
-@SuppressWarnings({ "PMD.DataflowAnomalyAnalysis", "PMD.BeanMembersShouldSerialize", "unchecked", "rawtypes" })
+@SuppressWarnings({ "PMD.DataflowAnomalyAnalysis", "PMD.BeanMembersShouldSerialize",
+                    "PMD.UnusedLocalVariable", "unchecked", "rawtypes" })
 public class OpenFeatureClient implements Client {
 
     private final OpenFeatureAPI openfeatureApi;