You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2019/09/04 15:26:34 UTC
[phoenix] branch master updated: PHOENIX-5428 Upgrade
maven-checkstyle-plugin version
This is an automated email from the ASF dual-hosted git repository.
elserj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/master by this push:
new c60b91a PHOENIX-5428 Upgrade maven-checkstyle-plugin version
c60b91a is described below
commit c60b91a431bfd2184e861c909136bb4c17c5b91b
Author: Istvan Toth <st...@stoty.hu>
AuthorDate: Tue Aug 6 15:19:48 2019 +0200
PHOENIX-5428 Upgrade maven-checkstyle-plugin version
This upgrades maven-checkstyle-plugin to 3.1.0 and
updates the checker.xml config files to work with recent
checkstyle versions
Closes #565
Signed-off-by: Josh Elser <el...@apache.org>
---
.../src/main/config/checkstyle/checker.xml | 39 +++++++++-------------
pom.xml | 2 +-
src/main/config/checkstyle/checker.xml | 39 +++++++++-------------
3 files changed, 33 insertions(+), 47 deletions(-)
diff --git a/phoenix-tracing-webapp/src/main/config/checkstyle/checker.xml b/phoenix-tracing-webapp/src/main/config/checkstyle/checker.xml
index ecf3946..323aac0 100755
--- a/phoenix-tracing-webapp/src/main/config/checkstyle/checker.xml
+++ b/phoenix-tracing-webapp/src/main/config/checkstyle/checker.xml
@@ -57,7 +57,6 @@ limitations under the License.
<module name="FileTabCharacter"/>
<module name="TreeWalker">
- <property name="cacheFile" value="target/checkstyle-cachefile"/>
<!-- Checks for blocks. You know, those {}'s -->
<!-- See http://checkstyle.sf.net/config_blocks.html -->
@@ -90,10 +89,6 @@ limitations under the License.
</module>
<!-- Switch statements should be complete and with independent cases -->
<module name="FallThrough"/>
- <!-- For hadoop_yarn profile, some YARN exceptions aren't loading in checkstyle -->
- <module name="RedundantThrows">
- <property name="suppressLoadErrors" value="true" />
- </module>
<module name="SimplifyBooleanExpression"/>
<module name="SimplifyBooleanReturn"/>
<!-- Only one statement per line allowed -->
@@ -256,26 +251,24 @@ limitations under the License.
<!-- No extra whitespace around types -->
<module name="GenericWhitespace"/>
- <!-- Required for SuppressionCommentFilter below -->
- <module name="FileContentsHolder"/>
- </module>
+ <!-- Setup special comments to suppress specific checks from source files -->
+ <module name="SuppressionCommentFilter">
+ <property name="offCommentFormat" value="CHECKSTYLE\: stop ([\w\|]+)"/>
+ <property name="onCommentFormat" value="CHECKSTYLE\: resume ([\w\|]+)"/>
+ <property name="checkFormat" value="$1"/>
+ </module>
- <!-- Setup special comments to suppress specific checks from source files -->
- <module name="SuppressionCommentFilter">
- <property name="offCommentFormat" value="CHECKSTYLE\: stop ([\w\|]+)"/>
- <property name="onCommentFormat" value="CHECKSTYLE\: resume ([\w\|]+)"/>
- <property name="checkFormat" value="$1"/>
- </module>
+ <!-- Turn off all checks between OFF and ON -->
+ <module name="SuppressionCommentFilter">
+ <property name="offCommentFormat" value="CHECKSTYLE\: OFF"/>
+ <property name="onCommentFormat" value="CHECKSTYLE\: ON"/>
+ </module>
- <!-- Turn off all checks between OFF and ON -->
- <module name="SuppressionCommentFilter">
- <property name="offCommentFormat" value="CHECKSTYLE\: OFF"/>
- <property name="onCommentFormat" value="CHECKSTYLE\: ON"/>
+ <!-- Turn off checks for the next N lines. -->
+ <module name="SuppressWithNearbyCommentFilter">
+ <property name="commentFormat" value="CHECKSTYLE: +IGNORE (\d+)"/>
+ <property name="influenceFormat" value="$1"/>
+ </module>
</module>
- <!-- Turn off checks for the next N lines. -->
- <module name="SuppressWithNearbyCommentFilter">
- <property name="commentFormat" value="CHECKSTYLE: +IGNORE (\d+)"/>
- <property name="influenceFormat" value="$1"/>
- </module>
</module>
diff --git a/pom.xml b/pom.xml
index 2579079..19a597e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -385,7 +385,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
- <version>2.13</version>
+ <version>3.1.0</version>
<executions>
<execution>
<id>validate</id>
diff --git a/src/main/config/checkstyle/checker.xml b/src/main/config/checkstyle/checker.xml
index ecf3946..323aac0 100644
--- a/src/main/config/checkstyle/checker.xml
+++ b/src/main/config/checkstyle/checker.xml
@@ -57,7 +57,6 @@ limitations under the License.
<module name="FileTabCharacter"/>
<module name="TreeWalker">
- <property name="cacheFile" value="target/checkstyle-cachefile"/>
<!-- Checks for blocks. You know, those {}'s -->
<!-- See http://checkstyle.sf.net/config_blocks.html -->
@@ -90,10 +89,6 @@ limitations under the License.
</module>
<!-- Switch statements should be complete and with independent cases -->
<module name="FallThrough"/>
- <!-- For hadoop_yarn profile, some YARN exceptions aren't loading in checkstyle -->
- <module name="RedundantThrows">
- <property name="suppressLoadErrors" value="true" />
- </module>
<module name="SimplifyBooleanExpression"/>
<module name="SimplifyBooleanReturn"/>
<!-- Only one statement per line allowed -->
@@ -256,26 +251,24 @@ limitations under the License.
<!-- No extra whitespace around types -->
<module name="GenericWhitespace"/>
- <!-- Required for SuppressionCommentFilter below -->
- <module name="FileContentsHolder"/>
- </module>
+ <!-- Setup special comments to suppress specific checks from source files -->
+ <module name="SuppressionCommentFilter">
+ <property name="offCommentFormat" value="CHECKSTYLE\: stop ([\w\|]+)"/>
+ <property name="onCommentFormat" value="CHECKSTYLE\: resume ([\w\|]+)"/>
+ <property name="checkFormat" value="$1"/>
+ </module>
- <!-- Setup special comments to suppress specific checks from source files -->
- <module name="SuppressionCommentFilter">
- <property name="offCommentFormat" value="CHECKSTYLE\: stop ([\w\|]+)"/>
- <property name="onCommentFormat" value="CHECKSTYLE\: resume ([\w\|]+)"/>
- <property name="checkFormat" value="$1"/>
- </module>
+ <!-- Turn off all checks between OFF and ON -->
+ <module name="SuppressionCommentFilter">
+ <property name="offCommentFormat" value="CHECKSTYLE\: OFF"/>
+ <property name="onCommentFormat" value="CHECKSTYLE\: ON"/>
+ </module>
- <!-- Turn off all checks between OFF and ON -->
- <module name="SuppressionCommentFilter">
- <property name="offCommentFormat" value="CHECKSTYLE\: OFF"/>
- <property name="onCommentFormat" value="CHECKSTYLE\: ON"/>
+ <!-- Turn off checks for the next N lines. -->
+ <module name="SuppressWithNearbyCommentFilter">
+ <property name="commentFormat" value="CHECKSTYLE: +IGNORE (\d+)"/>
+ <property name="influenceFormat" value="$1"/>
+ </module>
</module>
- <!-- Turn off checks for the next N lines. -->
- <module name="SuppressWithNearbyCommentFilter">
- <property name="commentFormat" value="CHECKSTYLE: +IGNORE (\d+)"/>
- <property name="influenceFormat" value="$1"/>
- </module>
</module>