You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Vyacheslav Koptilin (Jira)" <ji...@apache.org> on 2021/01/21 12:51:00 UTC
[jira] [Commented] (IGNITE-14000) Ignite 3.0: Fix codestyle for cli
and cli-common packages
[ https://issues.apache.org/jira/browse/IGNITE-14000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17269277#comment-17269277 ]
Vyacheslav Koptilin commented on IGNITE-14000:
----------------------------------------------
Hello [~kgusakov],
I left a few comments on your PR. Please take a look.
> Ignite 3.0: Fix codestyle for cli and cli-common packages
> ---------------------------------------------------------
>
> Key: IGNITE-14000
> URL: https://issues.apache.org/jira/browse/IGNITE-14000
> Project: Ignite
> Issue Type: Bug
> Reporter: Alexey Goncharuk
> Assignee: Kirill Gusakov
> Priority: Major
> Labels: ignite-3, ignite-3-cli-tool
> Fix For: 3.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Currently, the listed modules do not conform Ignite Coding Giudelines [1]:
> * classes are poorly formatted (e.g. braces for one-line if/for statements, invalid spacing for fields and statements blocks)
> * classes lack proper javadocs
> * packages missing package-info.java
> * code contains TODOs without tickets (tickets either should be created or TODOs removed/fixed)
> * code contains commented code blocks
> * Some error messages should be fixed ("Fail to..." -> "Failed to...", remove usages of "Please")
> [1] https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines
> [2] https://cwiki.apache.org/confluence/display/IGNITE/Abbreviation+Rules
--
This message was sent by Atlassian Jira
(v8.3.4#803005)