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

Maxim Muzafarov commented on IGNITE-11277:
------------------------------------------

[~amashenkov],

Actually, I've thought about it too, but according to my knowledge TC have no 
opened source code, so it's a problem to find out what is the real issue is. I 
can just only assume that it is some kind of migration issue from 2017 to 2018 
TC version. 

This is the cause why I'm suggesting to use an open source plugin instead of 
proprietary software -- we can fix all issues by ourselves.

> Use maven plugin as default code style checker for project
> ----------------------------------------------------------
>
>                 Key: IGNITE-11277
>                 URL: https://issues.apache.org/jira/browse/IGNITE-11277
>             Project: Ignite
>          Issue Type: Task
>            Reporter: Maxim Muzafarov
>            Assignee: Maxim Muzafarov
>            Priority: Major
>              Labels: inspections
>             Fix For: 2.8
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Currently, {{[Inspections] Core suite}} [1] on TC doesn't work well enough. 
> The suite has a {{FAILED}} status for more than 2 months due to some issues 
> on TeamCity application [2]. It confuses most of the members of the Apache 
> Ignite community. 
> Moreover, this suite is no longer checks configured rules. For instance, in 
> the master branch, 11 {{Unused imports}} can be found (e.g. for 
> {{IgniteCachePutAllRestartTest} 
>  [3]).
> I think the maven-checkstyle-plugin should be used as the default code style 
> checker.
> _Advantages:_
> * An IDE agnostic way for code checks
> * Can be used with different CI and build tools
> * Executable from the command line
> * Single configuration
> [1] 
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_InspectionsCore&branch_IgniteTests24Java8=%3Cdefault%3E&tab=buildTypeStatusDiv
> [2] https://youtrack.jetbrains.com/issue/TW-58504
> [3] 
> https://github.com/apache/ignite/blob/master/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteCachePutAllRestartTest.java#L29



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to