Hyukjin Kwon created PIO-12:
-------------------------------

             Summary: Adding some basic Scala style rules missed in 
scalastyle-config.xml
                 Key: PIO-12
                 URL: https://issues.apache.org/jira/browse/PIO-12
             Project: PredictionIO
          Issue Type: Improvement
            Reporter: Hyukjin Kwon


It seems there are some missing basic style checking rules. For example,

1. Removing tabs:

{code}
  <check level="error" class="org.scalastyle.file.FileTabChecker" 
enabled="true"></check>
{code}

2. Checking license in the header:

{code}
  <check level="error" class="org.scalastyle.file.HeaderMatchesChecker" 
enabled="true">
    <parameters>
       <parameter name="header"><![CDATA[/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */]]></parameter>
    </parameters>
  </check>
{code}


3. Checking spaces around +

{code}
  <check level="error" 
class="org.scalastyle.scalariform.SpacesAfterPlusChecker" 
enabled="true"></check>
  <check level="error" 
class="org.scalastyle.scalariform.SpacesBeforePlusChecker" 
enabled="true"></check>
{code}

4. Check non-ascii nodes

{code}
  <level="error" class="org.scalastyle.scalariform.NonASCIICharacterChecker" 
enabled="true"></check>
{code}

5. A space after comment start (See, 
[SpaceAfterCommentStartChecker|http://www.scalastyle.org/rules-dev.html#org_scalastyle_scalariform_SpaceAfterCommentStartChecker])

{code}
  <check level="error" 
class="org.scalastyle.scalariform.SpaceAfterCommentStartChecker" 
enabled="true"></check>
{code}





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to