(maven-javadoc-plugin) branch master updated: Rephrase source code fix interactive messages for clarity (#390)

[el...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new e2cf9d26 Rephrase source code fix interactive messages for clarity (#390)
e2cf9d26 is described below

commit e2cf9d26c0a9b0118939b128c7fc4b48e33be960
Author: Elliotte Rusty Harold <el...@users.noreply.github.com>
AuthorDate: Tue Jun 3 18:31:34 2025 +0000

    Rephrase source code fix interactive messages for clarity (#390)
    
    * rewrite source code fix user messages for clarity
---
 .../maven/plugins/javadoc/AbstractFixJavadocMojo.java       | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/javadoc/AbstractFixJavadocMojo.java b/src/main/java/org/apache/maven/plugins/javadoc/AbstractFixJavadocMojo.java
index 1170d9ad..aa1dde74 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/AbstractFixJavadocMojo.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/AbstractFixJavadocMojo.java
@@ -569,7 +569,7 @@ public abstract class AbstractFixJavadocMojo extends AbstractMojo {
         }
 
         if (!settings.isInteractiveMode()) {
-            getLog().error("Maven is not attempt to interact with the user for input. "
+            getLog().error("Maven is not interacting with the user for input. "
                     + "Verify the <interactiveMode/> configuration in your settings.");
             return false;
         }
@@ -579,11 +579,12 @@ public abstract class AbstractFixJavadocMojo extends AbstractMojo {
         getLog().warn("");
         getLog().warn("All warranties with regard to this Maven goal are disclaimed!");
         getLog().warn("The changes will be done directly in the source code.");
-        getLog().warn("The Maven Team strongly recommends the use of a SCM software BEFORE executing this goal.");
+        getLog().warn(
+                        "The Maven Team strongly recommends committing the code to source code management BEFORE executing this goal.");
         getLog().warn("");
 
         while (true) {
-            getLog().info("Are you sure to proceed? [Y]es [N]o");
+            getLog().info("Are you sure you want to proceed? [Y]es [N]o");
 
             try {
                 String userExpression = inputHandler.readLine();
@@ -592,7 +593,7 @@ public abstract class AbstractFixJavadocMojo extends AbstractMojo {
                     break;
                 }
                 if (JavadocUtil.equalsIgnoreCase(userExpression, "N", "No")) {
-                    getLog().info("No changes in your sources occur.");
+                    getLog().info("OK, I will not change your source code.");
                     return false;
                 }
             } catch (IOException e) {
@@ -737,9 +738,9 @@ public abstract class AbstractFixJavadocMojo extends AbstractMojo {
             }
         }
         if (clirrNewClasses.isEmpty() && clirrNewMethods.isEmpty()) {
-            getLog().info("Clirr NOT found API differences.");
+            getLog().info("Clirr did NOT find any API differences.");
         } else {
-            getLog().info("Clirr found API differences, i.e. new classes/interfaces or methods.");
+            getLog().info("Clirr found API differences; e.g. new classes, interfaces, or methods.");
         }
     }