[
https://issues.apache.org/jira/browse/PIO-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15390500#comment-15390500
]
Hyukjin Kwon commented on PIO-12:
---------------------------------
I would like to work on this if five of them look okay to add.
> 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}
> <check 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)
