You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by el...@apache.org on 2025/02/17 18:34:11 UTC

(maven-changes-plugin) branch html created (now 3273f48)

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

elharo pushed a change to branch html
in repository https://gitbox.apache.org/repos/asf/maven-changes-plugin.git


      at 3273f48  Minor doc fixes

This branch includes the following new commits:

     new 3273f48  Minor doc fixes

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



(maven-changes-plugin) 01/01: Minor doc fixes

Posted by el...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 3273f48218dcfc173058aa17afea3d679bec47f8
Author: Elliotte Rusty Harold <el...@ibiblio.org>
AuthorDate: Mon Feb 17 13:34:03 2025 -0500

    Minor doc fixes
---
 .../java/org/apache/maven/plugins/changes/ChangesReport.java |  2 +-
 .../apache/maven/plugins/changes/ChangesValidatorMojo.java   | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/changes/ChangesReport.java b/src/main/java/org/apache/maven/plugins/changes/ChangesReport.java
index ba98d6b..6531dd4 100644
--- a/src/main/java/org/apache/maven/plugins/changes/ChangesReport.java
+++ b/src/main/java/org/apache/maven/plugins/changes/ChangesReport.java
@@ -84,7 +84,7 @@ public class ChangesReport extends AbstractChangesReport {
     private File filteredOutputDirectory;
 
     /**
-     * applying filtering filtering "a la" resources plugin
+     * apply filtering "a la" resources plugin
      *
      * @since 2.2
      */
diff --git a/src/main/java/org/apache/maven/plugins/changes/ChangesValidatorMojo.java b/src/main/java/org/apache/maven/plugins/changes/ChangesValidatorMojo.java
index cb2849c..6660686 100644
--- a/src/main/java/org/apache/maven/plugins/changes/ChangesValidatorMojo.java
+++ b/src/main/java/org/apache/maven/plugins/changes/ChangesValidatorMojo.java
@@ -32,7 +32,7 @@ import org.apache.maven.plugins.changes.schema.XmlValidationHandler;
 import org.xml.sax.SAXParseException;
 
 /**
- * Goal which validate the <code>changes.xml</code> file.
+ * Goal which validates the <code>changes.xml</code> file.
  *
  * @author Olivier Lamy
  * @version $Id$
@@ -48,13 +48,13 @@ public class ChangesValidatorMojo extends AbstractChangesMojo {
     private String changesXsdVersion;
 
     /**
-     * Mojo failure if validation failed. If not and validation failed, only a warning will be logged.
+     * Mojo failure if validation failed. If false and validation failed, only a warning will be logged.
      */
     @Parameter(property = "changes.validate.failed", defaultValue = "false")
     private boolean failOnError;
 
     /**
-     * The path of the <code>changes.xml</code> file that will be converted into an HTML report.
+     * The path of the <code>changes.xml</code> file that will be validated.
      */
     @Parameter(property = "changes.xmlPath", defaultValue = "src/changes/changes.xml")
     private File xmlPath;
@@ -87,7 +87,7 @@ public class ChangesValidatorMojo extends AbstractChangesMojo {
                     logSchemaValidation(xmlValidationHandler.getErrors());
                     if (failOnError) {
                         throw new MojoExecutionException("changes.xml file " + xmlPath.getAbsolutePath()
-                                + " is not valid, see previous errors.");
+                                + " is not valid. See previous errors.");
                     } else {
                         getLog().info(" skip previous validation errors due to failOnError=false.");
                     }
@@ -95,7 +95,7 @@ public class ChangesValidatorMojo extends AbstractChangesMojo {
             } catch (SchemaValidatorException e) {
                 if (failOnError) {
                     throw new MojoExecutionException(
-                            "failed to validate changes.xml file " + xmlPath.getAbsolutePath() + ": " + e.getMessage(),
+                            "changes.xml file is not valid: " + xmlPath.getAbsolutePath() + ": " + e.getMessage(),
                             e);
                 }
             }
@@ -103,7 +103,7 @@ public class ChangesValidatorMojo extends AbstractChangesMojo {
     }
 
     private void logSchemaValidation(List<SAXParseException> errors) {
-        getLog().warn("failed to validate changes.xml file " + xmlPath.getAbsolutePath());
+        getLog().warn("changes.xml file is not valid: " + xmlPath.getAbsolutePath());
         getLog().warn("validation errors: ");
         for (SAXParseException error : errors) {
             getLog().warn(error.getMessage());