I don’t know what exactly mean in ‘compile time’. Maven indeed to it with Checkstyle.xml in when compile. but we only check 'import with star' and 'unused import’ in fe now. We could do more than that such as import order, tabs and intents, comments style and etc.. We can provide code style scheme for ides to format code automatically after we add rules to checkstyle.xml
> 2022年4月14日 17:02，ling miao <lingm...@apache.org> 写道： > > It is best to check the code style at compile time. > > Ling Miao > > morrysnow <morrys...@126.com> 于2022年4月14日周四 16:37写道： > >> 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
