diff options
author | Kousuke Saruta <sarutak@oss.nttdata.co.jp> | 2016-01-14 10:43:39 -0800 |
---|---|---|
committer | Josh Rosen <joshrosen@databricks.com> | 2016-01-14 10:43:39 -0800 |
commit | bcc7373f673d1a51b48fb95432ba5c4644dd5d23 (patch) | |
tree | cf5a9a97017c0abf13cae16b8cdbceea9937a3e9 | |
parent | 902667fd2766f0472a15851b1ed8fb5859593f97 (diff) | |
download | spark-bcc7373f673d1a51b48fb95432ba5c4644dd5d23.tar.gz spark-bcc7373f673d1a51b48fb95432ba5c4644dd5d23.tar.bz2 spark-bcc7373f673d1a51b48fb95432ba5c4644dd5d23.zip |
[SPARK-12821][BUILD] Style checker should run when some configuration files for style are modified but any source files are not.
When running the `run-tests` script, style checkers run only when any source files are modified but they should run when configuration files related to style are modified.
Author: Kousuke Saruta <sarutak@oss.nttdata.co.jp>
Closes #10754 from sarutak/SPARK-12821.
-rwxr-xr-x | dev/run-tests.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/dev/run-tests.py b/dev/run-tests.py index 8726889cbc..795db0dcfb 100755 --- a/dev/run-tests.py +++ b/dev/run-tests.py @@ -529,9 +529,14 @@ def main(): run_apache_rat_checks() # style checks - if not changed_files or any(f.endswith(".scala") for f in changed_files): + if not changed_files or any(f.endswith(".scala") + or f.endswith("scalastyle-config.xml") + for f in changed_files): run_scala_style_checks() - if not changed_files or any(f.endswith(".java") for f in changed_files): + if not changed_files or any(f.endswith(".java") + or f.endswith("checkstyle.xml") + or f.endswith("checkstyle-suppressions.xml") + for f in changed_files): # run_java_style_checks() pass if not changed_files or any(f.endswith(".py") for f in changed_files): |