aboutsummaryrefslogtreecommitdiff
path: root/pom.xml
diff options
context:
space:
mode:
authorDongjoon Hyun <dongjoon@apache.org>2016-04-09 21:31:20 -0700
committerReynold Xin <rxin@databricks.com>2016-04-09 21:31:20 -0700
commitaea30a1a9b79eb13d362ef32e4e9c8233e29f3dc (patch)
treeffb17949bc62f63eae7f6331eded161703719061 /pom.xml
parent72e66bb270efa3dc55560a4b2657e065cfdf2ea5 (diff)
downloadspark-aea30a1a9b79eb13d362ef32e4e9c8233e29f3dc.tar.gz
spark-aea30a1a9b79eb13d362ef32e4e9c8233e29f3dc.tar.bz2
spark-aea30a1a9b79eb13d362ef32e4e9c8233e29f3dc.zip
[SPARK-14465][BUILD] Checkstyle should check all Java files
## What changes were proposed in this pull request? Currently, `checkstyle` is configured to check the files under `src/main/java`. However, Spark has Java files in `src/main/scala`, too. This PR fixes the following configuration in `pom.xml` and the unchecked-so-far violations on those files. ```xml -<sourceDirectory>${basedir}/src/main/java</sourceDirectory> +<sourceDirectories>${basedir}/src/main/java,${basedir}/src/main/scala</sourceDirectories> ``` ## How was this patch tested? After passing the Jenkins build and manually `dev/lint-java`. (Note that Jenkins does not run `lint-java`) Author: Dongjoon Hyun <dongjoon@apache.org> Closes #12242 from dongjoon-hyun/SPARK-14465.
Diffstat (limited to 'pom.xml')
-rw-r--r--pom.xml2
1 files changed, 1 insertions, 1 deletions
diff --git a/pom.xml b/pom.xml
index 3f9e4abc32..4cbc6a2f11 100644
--- a/pom.xml
+++ b/pom.xml
@@ -2253,7 +2253,7 @@
<failOnViolation>false</failOnViolation>
<includeTestSourceDirectory>true</includeTestSourceDirectory>
<failOnWarning>false</failOnWarning>
- <sourceDirectory>${basedir}/src/main/java</sourceDirectory>
+ <sourceDirectories>${basedir}/src/main/java,${basedir}/src/main/scala</sourceDirectories>
<testSourceDirectory>${basedir}/src/test/java</testSourceDirectory>
<configLocation>dev/checkstyle.xml</configLocation>
<outputFile>${basedir}/target/checkstyle-output.xml</outputFile>