Merge pull request #181 from dcanar9/master

adding condition to excessive logging

Author: tatlax3636

pom.xml

@@ -4,7 +4,7 @@
     <artifactId>api-audit</artifactId>
     <packaging>jar</packaging>
     <name>${project.groupId}:${project.artifactId}</name>
-    <version>3.7.29-SNAPSHOT</version>
+    <version>3.7.30-SNAPSHOT</version>
     <description>Hygieia Audit Rest API Layer</description>
     <url>https://github.com/Hygieia/${repository.name}</url>
 

src/main/java/com/capitalone/dashboard/evaluator/CodeReviewEvaluator.java

@@ -221,8 +221,11 @@ private CodeReviewAuditResponseV2 getPeerReviewResponses(CollectorItem repoItem,
 
 
         Long startTime = System.currentTimeMillis();
+        if(notPeerReviewed.size() > 0) {
+            LOGGER.info(String.format("AutoMerge Check :: Reviewing %d Pull Request(s) with no peer review", notPeerReviewed.size()));
+        }
+
         // iterate though the gitRequests that failed the PR audit and check if the PR was auto merged
-        LOGGER.info(String.format("AutoMerge Check :: Reviewing %d Pull Request(s) with no peer review", notPeerReviewed.size()));
         for(GitRequest noPR: notPeerReviewed){
             boolean foundCommit = false;
 
@@ -264,8 +267,10 @@ private CodeReviewAuditResponseV2 getPeerReviewResponses(CollectorItem repoItem,
                 if(foundCommit){break;}     // stop iterating through PRs if commit was verified
             }
         }
-        LOGGER.info(String.format("AutoMerge Check :: Completed in %d ms", System.currentTimeMillis()-startTime));
 
+        if(notPeerReviewed.size() > 0) {
+            LOGGER.info(String.format("AutoMerge Check :: Completed in %d ms", System.currentTimeMillis() - startTime));
+        }
         //check any commits not directly tied to pr
         commits.stream().filter(commit -> !allPrCommitShas.contains(commit.getScmRevisionNumber()) && StringUtils.isEmpty(commit.getPullNumber()) && commit.getType() == CommitType.New).forEach(reviewAuditResponseV2::addDirectCommit);