[maven-checkstyle-plugin] 01/01: [MCHECKSTYLE-387] emit a warning when using an old version of checkstyle.

[eolivelli]

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 <bmarw...@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 )
         {