Re: [DISCUSS] Enable AvoidStarImport in Checkstyle

[Tsz Wo Sze]

+1. Thanks for working on this.

In general, let's put our energy on the code but do not spend much time on
the imports/whitespaces.  :)

Tsz-Wo

On Sat, Aug 8, 2020 at 8:58 PM runzhiwang <wangjie_challe...@qq.com> wrote:

> +1&nbsp;
>
>
> ------------------&nbsp;原始邮件&nbsp;------------------
> 发件人:
>                                                   "dev"
>                                                                 <
> amaliu...@apache.org&gt;;
> 发送时间:&nbsp;2020年8月9日(星期天) 中午11:47
> 收件人:&nbsp;"dev"<dev@ratis.apache.org&gt;;
>
> 主题:&nbsp;[DISCUSS] Enable AvoidStarImport in Checkstyle
>
>
>
> Hi community,
>
> Have you ever discussed why "AvoidStarImport" is not enabled? I noticed
> that there are some "import *" existing in the codebase and
> "AvoidStarImport" is not enabled in the checkstyle config. So I opened [1]
> to both enable it in config and fix "import *".
>
>
>
> [1]: https://github.com/apache/incubator-ratis/pull/173
>
> -Rui