[jira] [Assigned] (REEF-1055) Enable StyleCop.CSharp.ReadabilityRules and fix violations

["Mariia Mykhailova (JIRA)" <ji...@apache.org>]

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

Mariia Mykhailova reassigned REEF-1055:
---------------------------------------

    Assignee: Mariia Mykhailova

> Enable StyleCop.CSharp.ReadabilityRules and fix violations
> ----------------------------------------------------------
>
>                 Key: REEF-1055
>                 URL: https://issues.apache.org/jira/browse/REEF-1055
>             Project: REEF
>          Issue Type: Sub-task
>          Components: REEF.NET
>            Reporter: Mariia Mykhailova
>            Assignee: Mariia Mykhailova
>            Priority: Minor
>
> I suggest we adopt all rules, except:
> * SA1101: PrefixLocalCallsWithThis and SA1126: PrefixCallsCorrectly which require that we add {{this.}} whenever we reference class member. For similar issue in Java we decided against it.
> * SA1115: ParameterMustFollowComma which produces lots of violations which look like false positives (we don't have empty lines in lists of parametes).
> * SA1116: SplitParametersMustStartOnLineAfterDeclaration and SA1117: ParametersMustBeOnSameLineOrSeparateLines which increase code length for methods with multiple short parameters without increasing readability.
> The rest of the rules produce under 200 violations.



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