diff --git a/scalastyle-config.xml b/scalastyle-config.xml index 050c3f360476f7696281e1d163cbe0e409b596ed..dab1ebddc666e396d23a24ac905e8107ca8b648f 100644 --- a/scalastyle-config.xml +++ b/scalastyle-config.xml @@ -153,7 +153,7 @@ This file is divided into 3 sections: <check customId="visiblefortesting" level="error" class="org.scalastyle.file.RegexChecker" enabled="true"> <parameters><parameter name="regex">@VisibleForTesting</parameter></parameters> <customMessage><![CDATA[ - @VisibleForTesting auses classpath issues. Please note this in the java doc instead (SPARK-11615). + @VisibleForTesting causes classpath issues. Please note this in the java doc instead (SPARK-11615). ]]></customMessage> </check> @@ -203,6 +203,18 @@ This file is divided into 3 sections: <!-- Should turn this on, but we have a few places that need to be fixed first --> <check level="error" class="org.scalastyle.scalariform.EqualsHashCodeChecker" enabled="false"></check> + <!-- Import ordering. Currently warning only since there are lots of violations. --> + <check level="warning" class="org.scalastyle.scalariform.ImportOrderChecker" enabled="true"> + <parameters> + <parameter name="groups">java,scala,3rdParty,spark</parameter> + <parameter name="group.java">javax?\..+</parameter> + <parameter name="group.scala">scala\..+</parameter> + <parameter name="group.3rdParty">(?!org\.apache\.spark\.).*</parameter> + <parameter name="group.spark">org\.apache\.spark\..*</parameter> + </parameters> + </check> + + <!-- ================================================================================ --> <!-- rules we don't want --> <!-- ================================================================================ -->