You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2021/11/21 14:54:42 UTC

[maven-plugin-tools] branch master updated: [MPLUGIN-380] Always include thread safety in report

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 2ee1d60  [MPLUGIN-380] Always include thread safety in report
2ee1d60 is described below

commit 2ee1d603e2a71bcd99d58a5e6a4c4b2d2669d9c4
Author: Piotrek Żygieło <pz...@users.noreply.github.com>
AuthorDate: Sat Nov 20 23:06:30 2021 +0100

    [MPLUGIN-380] Always include thread safety in report
    
    This closes #51
---
 .../maven/tools/plugin/generator/PluginXdocGenerator.java   | 13 ++++++-------
 .../src/main/resources/pluginxdoc.properties                |  1 +
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator/PluginXdocGenerator.java b/maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator/PluginXdocGenerator.java
index ac064f4..c17ec59 100644
--- a/maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator/PluginXdocGenerator.java
+++ b/maven-plugin-tools-generators/src/main/java/org/apache/maven/tools/plugin/generator/PluginXdocGenerator.java
@@ -314,13 +314,12 @@ public class PluginXdocGenerator
             }
         }
 
-        if ( mojoDescriptor.isThreadSafe() )
-        {
-            addedUl = addUl( w, addedUl );
-            w.startElement( "li" );
-            w.writeMarkup( getString( "pluginxdoc.mojodescriptor.threadSafe" ) );
-            w.endElement(); //li
-        }
+        addedUl = addUl( w, addedUl );
+        w.startElement( "li" );
+        w.writeMarkup( getString( mojoDescriptor.isThreadSafe()
+                ? "pluginxdoc.mojodescriptor.threadSafe"
+                : "pluginxdoc.mojodescriptor.notThreadSafe" ) );
+        w.endElement(); //li
 
         value = mojoDescriptor.getSince();
         if ( StringUtils.isNotEmpty( value ) )
diff --git a/maven-plugin-tools-generators/src/main/resources/pluginxdoc.properties b/maven-plugin-tools-generators/src/main/resources/pluginxdoc.properties
index 273fe2b..17abbac 100644
--- a/maven-plugin-tools-generators/src/main/resources/pluginxdoc.properties
+++ b/maven-plugin-tools-generators/src/main/resources/pluginxdoc.properties
@@ -60,3 +60,4 @@ pluginxdoc.mojodescriptor.optionalParameters=Optional Parameters
 pluginxdoc.mojodescriptor.parameters=Parameters
 pluginxdoc.mojodescriptor.noParameter=(no parameters)
 pluginxdoc.mojodescriptor.threadSafe=The goal is thread-safe and supports parallel builds.
+pluginxdoc.mojodescriptor.notThreadSafe=The goal is not marked as thread-safe and thus does not support parallel builds.