Made changes based on PR feedback

etor/src/main/java/gov/hhs/cdc/trustedintermediary/etor/ruleengine/FhirResource.java

@@ -1,5 +1,11 @@
 package gov.hhs.cdc.trustedintermediary.etor.ruleengine;
 
+/**
+ * This interface represents a FHIR resource. It's used as a wrapper to decouple dependency on third
+ * party libraries.
+ *
+ * @param <T> the type of the underlying resource
+ */
 public interface FhirResource<T> {
     T getUnderlyingResource();
 }

etor/src/main/java/gov/hhs/cdc/trustedintermediary/etor/ruleengine/RuleLoader.java

@@ -1,6 +1,5 @@
 package gov.hhs.cdc.trustedintermediary.etor.ruleengine;
 
-import gov.hhs.cdc.trustedintermediary.wrappers.Logger;
 import gov.hhs.cdc.trustedintermediary.wrappers.formatter.Formatter;
 import gov.hhs.cdc.trustedintermediary.wrappers.formatter.FormatterProcessingException;
 import gov.hhs.cdc.trustedintermediary.wrappers.formatter.TypeReference;
@@ -13,7 +12,6 @@
 public class RuleLoader {
     private static final RuleLoader INSTANCE = new RuleLoader();
     @Inject Formatter formatter;
-    @Inject Logger logger;
 
     private RuleLoader() {}
 

etor/src/main/java/gov/hhs/cdc/trustedintermediary/etor/ruleengine/ValidationRule.java

@@ -20,6 +20,10 @@ public class ValidationRule implements Rule {
     private List<String> conditions;
     private List<String> validations;
 
+    /**
+     * Do not delete this constructor! It is used for JSON deserialization when loading rules from a
+     * file.
+     */
     public ValidationRule() {}
 
     public ValidationRule(

etor/src/main/java/gov/hhs/cdc/trustedintermediary/external/hapi/HapiFhirResource.java

@@ -3,6 +3,7 @@
 import gov.hhs.cdc.trustedintermediary.etor.ruleengine.FhirResource;
 import org.hl7.fhir.instance.model.api.IBaseResource;
 
+/** An implementation of {@link FhirResource} to use as a wrapper around HAPI FHIR IBaseResource */
 public class HapiFhirResource implements FhirResource<IBaseResource> {
 
     private final IBaseResource innerResource;