[ 
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}
<suppressions>
    <suppress checks="EmptyLineSeparator|MethodParamPad|SingleSpaceSeparator"
              files="BCrypt\.java|ConcurrentLinkedDeque8\.java"/>
    <suppress checks="NoWhitespaceBefore" 
files="ConcurrentLinkedHashMap\.java"/>
</suppressions>
{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)

Reply via email to