git-commit-id/git-commit-id-maven-plugin#624: downgrade some log messages from info to debug; Jgit is way chattier than the native git implementation, by downgrading we somewhat make them more equal

Author: TheSnoozer

src/main/java/pl/project13/core/jgit/DescribeCommand.java

@@ -114,7 +114,7 @@ private DescribeCommand(@Nonnull String evaluateOnCommit, @Nonnull Repository re
   @Nonnull
   public DescribeCommand always(boolean always) {
     this.alwaysFlag = always;
-    log.info(String.format("--always = %s", always));
+    log.debug(String.format("--always = %s", always));
     return this;
   }
 
@@ -136,7 +136,7 @@ public DescribeCommand always(boolean always) {
   public DescribeCommand forceLongFormat(@Nullable Boolean forceLongFormat) {
     if (forceLongFormat != null && forceLongFormat) {
       this.forceLongFormat = true;
-      log.info(String.format("--long = %s", true));
+      log.debug(String.format("--long = %s", true));
     }
     return this;
   }
@@ -161,7 +161,7 @@ public DescribeCommand abbrev(@Nullable Integer n) {
       if (n < 0) {
         throw new IllegalArgumentException("N (commit abbrev length) must be positive! (Was [" + n + "])");
       }
-      log.info(String.format("--abbrev = %s", n));
+      log.debug(String.format("--abbrev = %s", n));
       abbrev = n;
     }
     return this;
@@ -202,7 +202,7 @@ public DescribeCommand abbrev(@Nullable Integer n) {
   public DescribeCommand tags(@Nullable Boolean includeLightweightTagsInSearch) {
     if (includeLightweightTagsInSearch != null && includeLightweightTagsInSearch) {
       tagsFlag = includeLightweightTagsInSearch;
-      log.info(String.format("--tags = %s", includeLightweightTagsInSearch));
+      log.debug(String.format("--tags = %s", includeLightweightTagsInSearch));
     }
     return this;
   }
@@ -246,7 +246,7 @@ public DescribeCommand apply(@Nullable GitDescribeConfig config) {
   @Nonnull
   public DescribeCommand dirty(@Nullable String dirtyMarker) {
     Optional<String> option = Optional.ofNullable(dirtyMarker);
-    log.info(String.format("--dirty = %s", option.orElse("")));
+    log.debug(String.format("--dirty = %s", option.orElse("")));
     this.dirtyOption = option;
     return this;
   }
@@ -262,7 +262,7 @@ public DescribeCommand dirty(@Nullable String dirtyMarker) {
   public DescribeCommand match(@Nullable String pattern) {
     if (!"*".equals(pattern)) {
       matchOption = Optional.ofNullable(pattern);
-      log.info(String.format("--match = %s", matchOption.orElse("")));
+      log.debug(String.format("--match = %s", matchOption.orElse("")));
     }
     return this;
   }

src/main/java/pl/project13/core/jgit/JGitCommon.java

@@ -124,7 +124,7 @@ protected String createMatchPattern(String pattern) {
   protected Map<ObjectId, List<String>> findTagObjectIds(@Nonnull Repository repo, boolean includeLightweightTags, String matchPattern) {
     Map<ObjectId, List<DatedRevTag>> commitIdsToTags = getCommitIdsToTags(repo, includeLightweightTags, matchPattern);
     Map<ObjectId, List<String>> commitIdsToTagNames = transformRevTagsMapToDateSortedTagNames(commitIdsToTags);
-    log.info(String.format("Created map: [%s]", commitIdsToTagNames));
+    log.debug(String.format("Created map: [%s]", commitIdsToTagNames));
 
     return commitIdsToTagNames;
   }
@@ -137,7 +137,7 @@ protected RevCommit findEvalCommitObjectId(@Nonnull String evaluateOnCommit, @No
         RevCommit evalCommit = walk.parseCommit(evalCommitId);
         walk.dispose();
 
-        log.info(String.format("evalCommit is [%s]", evalCommit.getName()));
+        log.debug(String.format("evalCommit is [%s]", evalCommit.getName()));
         return evalCommit;
       }
     } catch (IOException ex) {
@@ -153,7 +153,7 @@ protected Map<ObjectId, List<DatedRevTag>> getCommitIdsToTags(@Nonnull Repositor
 
       List<Ref> tagRefs = Git.wrap(repo).tagList().call();
       Pattern regex = Pattern.compile(matchPattern);
-      log.info(String.format("Tag refs [%s]", tagRefs));
+      log.debug(String.format("Tag refs [%s]", tagRefs));
 
       for (Ref tagRef : tagRefs) {
         walk.reset();
@@ -168,7 +168,7 @@ protected Map<ObjectId, List<DatedRevTag>> getCommitIdsToTags(@Nonnull Repositor
         try {
           final RevTag revTag = walk.parseTag(resolvedCommitId);
           ObjectId taggedCommitId = revTag.getObject().getId();
-          log.info(String.format("Resolved tag [%s] [%s], points at [%s] ", revTag.getTagName(), revTag.getTaggerIdent(), taggedCommitId));
+          log.debug(String.format("Resolved tag [%s] [%s], points at [%s] ", revTag.getTagName(), revTag.getTaggerIdent(), taggedCommitId));
 
           // sometimes a tag, may point to another tag, so we need to unpack it
           while (isTagId(taggedCommitId)) {
@@ -185,7 +185,7 @@ protected Map<ObjectId, List<DatedRevTag>> getCommitIdsToTags(@Nonnull Repositor
           // it's an lightweight tag! (yeah, really)
           if (includeLightweightTags) {
             // --tags means "include lightweight tags"
-            log.info(String.format("Including lightweight tag [%s]", name));
+            log.debug(String.format("Including lightweight tag [%s]", name));
 
             DatedRevTag datedRevTag = new DatedRevTag(resolvedCommitId, name);
 
@@ -201,7 +201,7 @@ protected Map<ObjectId, List<DatedRevTag>> getCommitIdsToTags(@Nonnull Repositor
       }
 
       for (Map.Entry<ObjectId, List<DatedRevTag>> entry : commitIdsToTags.entrySet()) {
-        log.info(String.format("key [%s], tags => [%s] ", entry.getKey(), entry.getValue()));
+        log.debug(String.format("key [%s], tags => [%s] ", entry.getKey(), entry.getValue()));
       }
       return commitIdsToTags;
     } catch (Exception e) {