[maven-plugin-tools] 01/01: [MPLUGIN-452] Maven scope and module name warning should WARN

[cstamas]

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

cstamas pushed a commit to branch MPLUGIN-452-error-fails-the-build
in repository https://gitbox.apache.org/repos/asf/maven-plugin-tools.git

commit 1ce2b6e8f9d0c8786430a9ffbad3aae93e83aa41
Author: Tamas Cservenak <ta...@cservenak.net>
AuthorDate: Thu Jan 12 17:01:53 2023 +0100

    [MPLUGIN-452] Maven scope and module name warning should WARN
    
    Should be logged at WARN level, and not ERROR level. When
    Mojo logs ERROR, it is expected that it fails the build.
    And this also confuses the Verifier.
    
    ---
    
    https://issues.apache.org/jira/browse/MPLUGIN-452
---
 .../java/org/apache/maven/plugin/plugin/DescriptorGeneratorMojo.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/DescriptorGeneratorMojo.java
 
b/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/DescriptorGeneratorMojo.java
index ba836830..307d5336 100644
--- 
a/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/DescriptorGeneratorMojo.java
+++ 
b/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/DescriptorGeneratorMojo.java
@@ -301,7 +301,7 @@ public class DescriptorGeneratorMojo
                         && project.getArtifactId().toLowerCase().endsWith( 
"-plugin" )
                         && !"org.apache.maven.plugins".equals( 
project.getGroupId() ) )
         {
-            getLog().error( LS + LS + "Artifact Ids of the format 
maven-___-plugin are reserved for" + LS
+            getLog().warn( LS + LS + "Artifact Ids of the format 
maven-___-plugin are reserved for" + LS
                                 + "plugins in the Group Id 
org.apache.maven.plugins" + LS
                                 + "Please change your artifactId to the format 
___-maven-plugin" + LS
                                 + "In the future this error will break the 
build." + LS + LS );
@@ -330,7 +330,7 @@ public class DescriptorGeneratorMojo
                 }
                 errorMessage.append( LS ).append( LS );
 
-                getLog().error( errorMessage.toString() );
+                getLog().warn( errorMessage.toString() );
             }
         }