Yeah, it will influence many files. The good news is there are not too many Scala files compared with Java files. There are indeed some problems to rebase code from fork or PRs. What we have done for Java code is: First, rebasing to the commit which adds the spotless config, second applying the spotless locally, then rebasing to the master latest commit.
> It also hides other commits when I do `Git Blame`. This should be acceptable because it is basically a problem of spaces and line length. Thanks, Xianyang ________________________________ 发件人: Manu Zhang <owenzhang1...@gmail.com> 发送时间: 2023年7月19日 0:59 收件人: dev@iceberg.apache.org <dev@iceberg.apache.org> 主题: Re: Discussion about applying spotless for scala code Is this the right time to do so? It will bring a lot of conflicts with open PRs and forked codebases. It also hides other commits when I do `Git Blame`. My two cents, Manu On Wed, Jul 19, 2023 at 12:24 AM Liu Xianyang <liu-xiany...@hotmail.com<mailto:liu-xiany...@hotmail.com>> wrote: Hello everyone: I'd like the raise a discussion about applying spotless for Scala code. We need to get more advice since it will change a lot of files. We have already applied spotless for Java code which helps developers easily follow the code style. And also it makes better code management. I see there are already some discussions about this: https://github.com/apache/iceberg/issues/6736. I have submitted a PR for this: https://github.com/apache/iceberg/pull/8023. Hopeful get more advice from you. Thanks, Xianyang