You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2018/09/05 13:24:54 UTC

[flink] 01/04: [hotfix][checkstyle] Enable checkstyle for flink-core/test.util

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

chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git

commit e210a88f08e6cfde5a9bc14d8282ca23a32eb03d
Author: zentol <ch...@apache.org>
AuthorDate: Wed Sep 5 15:03:32 2018 +0200

    [hotfix][checkstyle] Enable checkstyle for flink-core/test.util
---
 tools/maven/suppressions-core.xml | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/tools/maven/suppressions-core.xml b/tools/maven/suppressions-core.xml
index db5d16a..e44060e 100644
--- a/tools/maven/suppressions-core.xml
+++ b/tools/maven/suppressions-core.xml
@@ -103,11 +103,6 @@ under the License.
 		files="(.*)test[/\\](.*)types[/\\](.*)"
 		checks="AvoidStarImport|NeedBraces"/>
 
-	<!--Only additional checks for test sources. Those checks were present in the "pre-strict" checkstyle but were not applied to test sources. We do not want to suppress them for sources directory-->
-	<suppress
-		files="(.*)test[/\\](.*)util[/\\](.*)"
-		checks="UnusedImports|AvoidStarImport"/>
-
 	<suppress
 		files="(.*)test[/\\](.*)testutils[/\\](.*)"
 		checks="NewlineAtEndOfFile|RegexpSingleline|TodoComment|RedundantImport|ImportOrder|RedundantModifier|JavadocMethod|JavadocParagraph|JavadocType|JavadocStyle|PackageName|TypeNameCheck|ConstantNameCheck|StaticVariableNameCheck|MemberNameCheck|MethodNameCheck|ParameterName|LocalFinalVariableName|LocalVariableName|LeftCurly|UpperEll|FallThrough|reliefPattern|SimplifyBooleanExpression|EmptyStatement|ModifierOrder|EmptyLineSeparator|WhitespaceAround|WhitespaceAfter|NoWhitespaceAfter|NoWhite [...]