(maven-scm) branch master updated: Throw is enough (#1306)

[elharo]

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-scm.git


The following commit(s) were added to refs/heads/master by this push:
     new 5946906d3 Throw is enough (#1306)
5946906d3 is described below

commit 5946906d31d81a5d8b47868647669ef410205744
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Tue Aug 19 10:32:37 2025 +0000

    Throw is enough (#1306)
---
 .../java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java    | 1 -
 maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java  | 1 -
 2 files changed, 2 deletions(-)

diff --git 
a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
 
b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
index 09bf2be6a..0e6a62602 100644
--- 
a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
+++ 
b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
@@ -76,7 +76,6 @@ public void execute() throws MojoExecutionException {
             }
 
             if (!result.getChangedFiles().isEmpty()) {
-                getLog().error(errorMessage);
                 throw new MojoExecutionException(errorMessage);
             }
         } catch (IOException | ScmException e) {
diff --git 
a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java 
b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
index 622ce3856..eaeb41ea9 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
@@ -140,7 +140,6 @@ public void execute() throws MojoExecutionException {
                     getLog().info("Using timestamp '" + tagTimestamp + "'");
                 } catch (IllegalArgumentException e) {
                     String msg = "The timestamp format '" + timestampFormat + 
"' is invalid.";
-                    getLog().error(msg, e);
                     throw new MojoExecutionException(msg, e);
                 }