[maven-checkstyle-plugin] branch MCHECKSTYLE-387 created (now b8e77df)

[eo...@apache.org]

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

eolivelli pushed a change to branch MCHECKSTYLE-387
in repository https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin.git.


      at b8e77df  [MCHECKSTYLE-387] emit a warning when using an old version of checkstyle.

This branch includes the following new commits:

     new b8e77df  [MCHECKSTYLE-387] emit a warning when using an old version of checkstyle.

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-checkstyle-plugin] 01/01: [MCHECKSTYLE-387] emit a warning when using an old version of checkstyle.

[eo...@apache.org]

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

eolivelli pushed a commit to branch MCHECKSTYLE-387
in repository https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin.git

commit b8e77df4691522970f6ccdcf6eb8362495312407
Author: Benjamin Marwell <bm...@gmail.com>
AuthorDate: Thu Feb 27 06:51:54 2020 +0100

    [MCHECKSTYLE-387] emit a warning when using an old version of checkstyle.
---
 .../maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java   | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/main/java/org/apache/maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java b/src/main/java/org/apache/maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java
index ce7f213..1a8ca2e 100644
--- a/src/main/java/org/apache/maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java
+++ b/src/main/java/org/apache/maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java
@@ -314,6 +314,13 @@ public class DefaultCheckstyleExecutor
         try
         {
             checker.setClassLoader( projectClassLoader );
+            /*
+             * MCHECKSTYLE-387: If the previous method call was successful, emit a warning that the user is using
+             * an old version of checkstyle.
+             */
+            getLogger().warn( "Old version of checkstyle detected. Consider updating to >= v8.30" );
+            getLogger().warn( "For more information see: "
+                + "https://maven.apache.org/plugins/maven-checkstyle-plugin/examples/upgrading-checkstyle.html" );
         }
         catch ( NoSuchMethodError ignored )
         {