great, I will trim it in my PR.

checkstyle.xml need more discussion

On Thu, Apr 14, 2022 at 5:48 PM morrysnow <morrys...@126.com> wrote:

> I agree with u.
>
> For the first point, I want to list rules first, and then change
> checkstyle.xml.
>
> For the second point, original change information in git is not lost, we
> just need to do blame on the version prior to the ‘code style’ commit and
> some gui tools could list history of one file. Anyway, it is not very
> convenience.
>
> > 2022年4月14日 17:35,Shuo Wang <wangshuo...@gmail.com> 写道:
> >
> > In general, I believe that we should do some work to make the code clean
> > and readable.
> >
> > My concern is:
> > 1. We should have an agreement on the code style specification in the
> > community at first.
> > 2. If the many lines of code change after applying the code style rule,
> we
> > would lose the original changelog via `git blame`.
> >
> > vin jake <jakevin...@gmail.com> 于2022年4月14日周四 17:12写道:
> >
> >> I have add it in PR https://github.com/apache/incubator-doris/pull/8987
> >>
> >> On Thu, Apr 14, 2022 at 4:37 PM morrysnow <morrys...@126.com> wrote:
> >>
> >>> Hi, devs,
> >>>
> >>> Currently, we only have two rules in checkstyle.xml in fe. These are
> all
> >>> about import. So, the code style in fe is very casual.
> >>> I want to add more rules to checkstyle.xml in fe to Improve code
> >>> readability, and adjust all fe code to satisfy new code style step by
> >> step.
> >>> What do you think about it? If this is a good idea. I will research
> which
> >>> rules apply to our code and put together a list.
> >>>
> >>> ---------------------------------------------------------------------
> >>> To unsubscribe, e-mail: dev-unsubscr...@doris.apache.org
> >>> For additional commands, e-mail: dev-h...@doris.apache.org
> >>>
> >>>
> >>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@doris.apache.org
> For additional commands, e-mail: dev-h...@doris.apache.org
>
>

Reply via email to