spark-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prashant Sharma (JIRA)" <j...@apache.org>
Subject [jira] [Assigned] (SPARK-1338) Create Additional Style Rules
Date Wed, 09 Apr 2014 09:35:14 GMT

     [ https://issues.apache.org/jira/browse/SPARK-1338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Prashant Sharma reassigned SPARK-1338:
--------------------------------------

    Assignee: Prashant Sharma  (was: prashant)

> Create Additional Style Rules
> -----------------------------
>
>                 Key: SPARK-1338
>                 URL: https://issues.apache.org/jira/browse/SPARK-1338
>             Project: Spark
>          Issue Type: Improvement
>          Components: Project Infra
>            Reporter: Patrick Cogan
>            Assignee: Prashant Sharma
>             Fix For: 1.0.0
>
>
> There are a few other rules that would be helpful to have. Also we should add tests for
these rules because it's easy to get them wrong. I gave some example comparisons from a javascript
style checker.
> Require spaces in type declarations:
> def foo:String = X // no
> def foo: String = XXX
> def x:Int = 100 // no
> val x: Int = 100
> Require spaces after keywords:
> if(x - 3) // no
> if (x + 10)
> See: requireSpaceAfterKeywords from
> https://github.com/mdevils/node-jscs
> Disallow spaces inside of parentheses:
> val x = ( 3 + 5 ) // no
> val x = (3 + 5)
> See: disallowSpacesInsideParentheses from
> https://github.com/mdevils/node-jscs
> Require spaces before and after binary operators:
> See: requireSpaceBeforeBinaryOperators
> See: disallowSpaceAfterBinaryOperators
> from https://github.com/mdevils/node-jscs



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message