You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Maxim Muzafarov (Jira)" <ji...@apache.org> on 2019/08/23 13:27:00 UTC

[jira] [Commented] (IGNITE-5705) Integrate Checkstyle into the Build Process

    [ https://issues.apache.org/jira/browse/IGNITE-5705?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16914264#comment-16914264 ] 

Maxim Muzafarov commented on IGNITE-5705:
-----------------------------------------

[~denizarsan], since the IGNITE-11277 checkstyle configuration has been merged to the master branch we can start supporting checkstyle rules one by one. Do you have any ideas about what the next rule we should support to? All the issues related to checkstyle-plugin mark with `checkstyle` label.
See the list below:
https://issues.apache.org/jira/browse/IGNITE-12077?jql=labels%20%3D%20checkstyle%20and%20project%20%3D%20Ignite

> Integrate Checkstyle into the Build Process
> -------------------------------------------
>
>                 Key: IGNITE-5705
>                 URL: https://issues.apache.org/jira/browse/IGNITE-5705
>             Project: Ignite
>          Issue Type: New Feature
>            Reporter: Deniz Arsan
>            Priority: Major
>              Labels: checkstyle
>         Attachments: checkstyle.xml, pom.xml.patch
>
>
> Hi, 
>  
> I see https://issues.apache.org/jira/browse/IGNITE-1236, and I wonder if you have made the Checkstyle configurations ready? (I failed to find any in the GitHub repository)
>  
> If not, attached is a Checkstyle configuration file that is consistent with the current code base  (the master branch passes {{mvn checkstyle:checkstyle}}). Also attached is the patch for pom.xml in parent that adds {{maven-checkstyle-plugin}}.
>  
> I can include more checks in the configuration if there are any specific requirements for coding style. 
>  
> Let me know.
>  
> Thanks!



--
This message was sent by Atlassian Jira
(v8.3.2#803003)