Skip to content

Commit

Permalink
[SPARK-12692][BUILD] Scala style: check no white space before comma a…
Browse files Browse the repository at this point in the history
…nd colon

We should not put a white space before `,` and `:` so let's check it.
Because there are lots of style violations, first, I'd like to add a checker, enable and let the level `warning`.
Then, I'd like to fix the style step by step.

Author: Kousuke Saruta <sarutak@oss.nttdata.co.jp>

Closes #10643 from sarutak/SPARK-12692.
  • Loading branch information
sarutak authored and rxin committed Jan 8, 2016
1 parent 726bd3c commit 794ea55
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions scalastyle-config.xml
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,12 @@ This file is divided into 3 sections:
</parameters>
</check>

<!-- Should turn this on, but we have a few places that need to be fixed first -->
<check level="warning" class="org.scalastyle.scalariform.DisallowSpaceBeforeTokenChecker" enabled="true">
<parameters>
<parameter name="tokens">COLON, COMMA</parameter>
</parameters>
</check>

<!-- ================================================================================ -->
<!-- rules we don't want -->
Expand Down

0 comments on commit 794ea55

Please sign in to comment.