diff options
author | Dongjoon Hyun <dongjoon@apache.org> | 2016-04-09 21:31:20 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2016-04-09 21:31:20 -0700 |
commit | aea30a1a9b79eb13d362ef32e4e9c8233e29f3dc (patch) | |
tree | ffb17949bc62f63eae7f6331eded161703719061 /README.md | |
parent | 72e66bb270efa3dc55560a4b2657e065cfdf2ea5 (diff) | |
download | spark-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 'README.md')
0 files changed, 0 insertions, 0 deletions