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 2017/05/28 06:17:51 UTC

[21/21] flink git commit: [FLINK-6711] Activate strict checkstyle for flink-connectors

[FLINK-6711] Activate strict checkstyle for flink-connectors

This closes #3992.


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/4f50dc4d
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/4f50dc4d
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/4f50dc4d

Branch: refs/heads/master
Commit: 4f50dc4df14d00203a03873edaf4252082ef4d38
Parents: b58545e
Author: zentol <ch...@apache.org>
Authored: Thu May 25 18:52:40 2017 +0200
Committer: zentol <ch...@apache.org>
Committed: Sun May 28 00:11:50 2017 +0200

----------------------------------------------------------------------
 flink-connectors/pom.xml | 39 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/4f50dc4d/flink-connectors/pom.xml
----------------------------------------------------------------------
diff --git a/flink-connectors/pom.xml b/flink-connectors/pom.xml
index 5af85b8..cbb48e0 100644
--- a/flink-connectors/pom.xml
+++ b/flink-connectors/pom.xml
@@ -101,5 +101,44 @@ under the License.
 			</modules>
 		</profile>
 	</profiles>
+	
+	<build>
+		<plugins>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-checkstyle-plugin</artifactId>
+				<version>2.17</version>
+				<dependencies>
+					<dependency>
+						<groupId>com.puppycrawl.tools</groupId>
+						<artifactId>checkstyle</artifactId>
+						<version>6.19</version>
+					</dependency>
+				</dependencies>
+				<configuration>
+					<configLocation>/tools/maven/strict-checkstyle.xml</configLocation>
+					<suppressionsLocation>/tools/maven/suppressions.xml</suppressionsLocation>
+					<includeTestSourceDirectory>true</includeTestSourceDirectory>
+					<logViolationsToConsole>true</logViolationsToConsole>
+					<failOnViolation>true</failOnViolation>
+				</configuration>
+				<executions>
+					<!--
+					Execute checkstyle after compilation but before tests.
+
+					This ensures that any parsing or type checking errors are from
+					javac, so they look as expected. Beyond that, we want to
+					fail as early as possible.
+					-->
+					<execution>
+						<phase>test-compile</phase>
+						<goals>
+							<goal>check</goal>
+						</goals>
+					</execution>
+				</executions>
+			</plugin>
+		</plugins>
+	</build>
 
 </project>