You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2018/04/01 12:52:27 UTC

[maven-site-plugin] branch master updated: [MSITE-815] highlight skin used to render site and documents statistics

This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new 53905d8  [MSITE-815] highlight skin used to render site and documents statistics
53905d8 is described below

commit 53905d8f1740a364e13e1ee6d2f98d8bcd26ea88
Author: Hervé Boutemy <hb...@apache.org>
AuthorDate: Sun Apr 1 14:50:22 2018 +0200

    [MSITE-815] highlight skin used to render site and documents statistics
---
 .../site/render/AbstractSiteRenderingMojo.java     |  6 +++--
 .../apache/maven/plugins/site/render/SiteMojo.java | 26 +++++++++++++++-------
 2 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
index 605dc09..7614d0c 100644
--- a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
+++ b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
@@ -62,6 +62,8 @@ import org.codehaus.plexus.personality.plexus.lifecycle.phase.Contextualizable;
 import org.codehaus.plexus.util.ReaderFactory;
 import org.codehaus.plexus.util.StringUtils;
 
+import static org.apache.maven.shared.utils.logging.MessageUtils.buffer;
+
 /**
  * Base class for site rendering mojos.
  *
@@ -339,7 +341,7 @@ public abstract class AbstractSiteRenderingMojo
         SiteRenderingContext context;
         if ( templateFile != null )
         {
-            getLog().info( "Rendering site with " + templateFile + " template file." );
+            getLog().info( buffer().strong( "Rendering site with " + templateFile + " template file." ).toString() );
 
             if ( !templateFile.exists() )
             {
@@ -355,7 +357,7 @@ public abstract class AbstractSiteRenderingMojo
                 Artifact skinArtifact =
                     siteTool.getSkinArtifactFromRepository( localRepository, repositories, decorationModel );
 
-                getLog().info( "Rendering site with " + skinArtifact.getId() + " skin." );
+                getLog().info( buffer().strong( "Rendering site with " + skinArtifact.getId() + " skin." ).toString() );
 
                 context = siteRenderer.createContextForSkin( skinArtifact, attributes, decorationModel,
                                                              project.getName(), locale );
diff --git a/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java b/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java
index 04ec499..ddaf418 100644
--- a/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java
+++ b/src/main/java/org/apache/maven/plugins/site/render/SiteMojo.java
@@ -42,6 +42,9 @@ import org.apache.maven.project.MavenProject;
 import org.apache.maven.reporting.MavenReport;
 import org.apache.maven.reporting.MavenReportException;
 import org.apache.maven.reporting.exec.MavenReportExecution;
+import org.apache.maven.shared.utils.logging.MessageBuilder;
+
+import static org.apache.maven.shared.utils.logging.MessageUtils.buffer;
 
 /**
  * Generates the site for a single project.
@@ -251,20 +254,27 @@ public class SiteMojo
 
         if ( doxiaDocuments.size() > 0 )
         {
-            StringBuilder sb = new StringBuilder( 15 * counts.size() );
+            MessageBuilder mb = buffer();
+            mb.a( "Rendering " );
+            mb.strong( doxiaDocuments.size() + ( generated ? " generated" : "" ) + " Doxia document"
+                + ( doxiaDocuments.size() > 1 ? "s" : "" ) );
+            mb.a( ": " );
+
+            boolean first = true;
             for ( Map.Entry<String, Integer> entry : counts.entrySet() )
             {
-                if ( sb.length() > 0 )
+                if ( first )
+                {
+                    first = false;
+                }
+                else
                 {
-                    sb.append( ", " );
+                    mb.a( ", " );
                 }
-                sb.append( entry.getValue() );
-                sb.append( ' ' );
-                sb.append( entry.getKey() );
+                mb.strong( entry.getValue() + " " + entry.getKey() );
             }
 
-            getLog().info( "Rendering " + doxiaDocuments.size() + ( generated ? " generated" : "" ) + " Doxia document"
-                + ( doxiaDocuments.size() > 1 ? "s" : "" ) + ": " + sb.toString() );
+            getLog().info( mb.toString() );
 
             siteRenderer.render( doxiaDocuments.values(), context, outputDir );
         }

-- 
To stop receiving notification emails like this one, please contact
hboutemy@apache.org.