Improve result log (#338)

SimonJPegg · Apr 16, 2024 · a9b155c · a9b155c
1 parent 9f83818
commit a9b155c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/main/java/org/antipathy/mvn_scalafmt/FormatMojo.java b/src/main/java/org/antipathy/mvn_scalafmt/FormatMojo.java
@@ -86,7 +86,7 @@ public void execute() throws MojoExecutionException {
                         project.getBasedir(),
                         useSpecifiedRepositories ? getRepositoriesUrls(mavenRepositories) : new ArrayList<String>()
                 ).format(sources);
-                getLog().info(result.toString());
+                result.print(getLog());
                 if (validateOnly && result.unformattedFiles() != 0) {
                     throw new MojoExecutionException("Scalafmt: Unformatted files found");
                 }

diff --git a/src/main/scala/org/antipathy/mvn_scalafmt/model/Summary.scala b/src/main/scala/org/antipathy/mvn_scalafmt/model/Summary.scala
@@ -1,5 +1,7 @@
 package org.antipathy.mvn_scalafmt.model
 
+import org.apache.maven.plugin.logging.Log
+
 // $COVERAGE-OFF$
 
 /** Class representing the result of a run of the plugin
@@ -13,11 +15,13 @@ case class Summary(
   fileDetails: Seq[FileSummary]
 ) {
 
-  override def toString: String =
-    s"""Scalafmt results: $unformattedFiles of $totalFiles were unformatted
-       |Details:
-       |${fileDetails.mkString(System.lineSeparator)}
-     """.stripMargin
+  def print(log: Log) = {
+    log.info(s"Scalafmt results: $unformattedFiles of $totalFiles were unformatted")
+    log.info("Details:")
+    fileDetails.foreach { fileSummary =>
+      log.info("- " + fileSummary.toString)
+    }
+  }
 }
 
 // $COVERAGE-ON$