You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Nikolay Izhikov (Jira)" <ji...@apache.org> on 2021/03/19 18:17:00 UTC

[jira] [Updated] (IGNITE-14361) Add WhiteSpace after checkstyle rule

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

Nikolay Izhikov updated IGNITE-14361:
-------------------------------------
    Description: 
Right now checkstyle plugin don't check whitespace after rule that allows the following violation of Ignite code style

{code:java}
finish.apply(p.id,msg.result(), msg.error());
 {code}

Need to add the following rule to prevent it:
{code:xml}
        <module name="WhitespaceAfter">
            <property name="tokens" value="COMMA, SEMI"/>
        </module>
{code}

  was:
Right now checkstyle plugin don't check whitespace after rule that allows the following violation of Ignite code style

{code:java}
finish.apply(p.id,msg.result(), msg.error());
 {code}

Need to add the following rule to prevent it:
{code:xml}
        <module name="WhitespaceAfter">
            <property name="tokens" value="COMMA, SEMI"/>
        </module>

{code}


> Add WhiteSpace after checkstyle rule
> ------------------------------------
>
>                 Key: IGNITE-14361
>                 URL: https://issues.apache.org/jira/browse/IGNITE-14361
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Nikolay Izhikov
>            Assignee: Nikolay Izhikov
>            Priority: Major
>
> Right now checkstyle plugin don't check whitespace after rule that allows the following violation of Ignite code style
> {code:java}
> finish.apply(p.id,msg.result(), msg.error());
>  {code}
> Need to add the following rule to prevent it:
> {code:xml}
>         <module name="WhitespaceAfter">
>             <property name="tokens" value="COMMA, SEMI"/>
>         </module>
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)