Hi, there is a new PR about format doris.
https://github.com/apache/incubator-doris/pull/9072

I think it's time to ensure what checkstyle rules  we should add for fe.

All people who care about format rule can put up your thoughts.

We can add it in the issue:
https://github.com/apache/incubator-doris/issues/8985


On Thu, Apr 14, 2022 at 6:52 PM vin jake <jakevin...@gmail.com> wrote:

> 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