+1
> 2022年8月30日 17:54,Chufeng Gao <chufeng...@gmail.com> 写道:
>
> Hi developers,
>
> Currently, ST uses the `Checkstyle` plugin for style and formatting checks.
> I propose that we replace `Checkstyle` with `Spotless`.
>
> Huge advantage of using `Spotless`[1][2] over `CheckStyle` is in addition
> to check the formatting of the code it also has `apply` goal that fixes all
> the style and formatting errors automatically with a single line of command
> `mvn spotless:apply`. `Spotless` does not rely on IDE so that
> new contributors do not have to spend time setting up their IDE with
> checkstyle.
> Furthermore, we could configure `git pre-commit hook` to
> trigger `Spotless` and in this way contributors no more need to worry about
> formatting and style issues as they get fixed automatically
> when contributors run `git commit xxx`.
>
> FYI, we have replaced `Checkstyle` with `Spotless` in the Apache
> DolphinScheduler project. You could refer to the related issues[3], PRs and
> discussions[4] for details.
>
> If you have any suggestions or concerns, you are very welcome to comment in
> this email thread or in issue #2492 [5]
>
>
> [1] https://github.com/diffplug/spotless
> [2] https://github.com/diffplug/spotless/tree/main/plugin-maven
> [3] https://github.com/apache/dolphinscheduler/issues/10963
> [4] https://lists.apache.org/thread/9sqvd1wlfgkyn2y926l732zj1n3z7xkx
> [5] https://github.com/apache/incubator-seatunnel/issues/2492
>
> Thanks!
>
> *Best Regards,*
>
> *Chufeng (Eric) Gao*