Yes. Mostly when I run `mvn clean package -DskipTests -DskipITs` locally, I saw some style issues. Two main things not well taken care of by the auto reformatting: - The import order. - Depending on the format (new line for the body), the lambda function may throw indentation errors.
Also, for some files, reformatting in IDE gives me a lot of changes, so sometimes I don't run reformatting to avoid confusion of those changes in the PR. I guess I need to be more proactive on cleaning up the code I'm touching. Best, - Ethan On Fri, Feb 28, 2020 at 2:33 PM Vinoth Chandar <vin...@apache.org> wrote: > Hey Ethan, > > Thanks for raising this up again. Before we go to fixes, may be it would > good to summarize the current problems one by one. We can then discuss > options? > > I have been using checkstyle files to format and it’s been tolerable. > Import order etc does not work. But we have turned it off. > > Thanks > Vinoth > > On Fri, Feb 28, 2020 at 12:47 PM Y Ethan Guo <ethan.guoyi...@gmail.com> > wrote: > > > Reviving this thread... > > > > I'm hitting checkstyle issues locally again and thinking that it might be > > worth trying GJF. It interoperates well with IntelliJ and the automated > > tool reformats the code in 1 shot as Minh suggested. > > > > If sweeping checkstyle fixes in the codebase disrupts development, we can > > temporarily disable checkstyle in this case and for each future PR, we > > generate a corresponding cleanup PR just for the changed files (maybe a > > script to autogen the clean PR?), to reformat code incrementally. > > > > wdyt? > > > > Best, > > - Ethan > > > > On Mon, Dec 23, 2019 at 6:24 PM lamber...@163.com <lamber...@163.com> > > wrote: > > > > > > > >
