+1
> 在 2022年7月15日,14:52,Chufeng Gao <[email protected]> 写道:
>
> Hi developers,
>
> Currently, DS uses `Checkstyle` plugin for style and formatting checks.
> However, there are tons of legacy formatting errors in the project popping
> out every time we build the project. Thus, in practice, some developers
> tend to use `-Dcheckstyle.skip` flag when building the project and this
> make the `style check` a little bit awkward.
>
> 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 automatically. With `Spotless`, we could fix the
> legacy formatting errors once for all and developers could fix new format
> errors with a single line of command `mvn spotless:apply`.
>
> If you have any suggestions, you are very welcome to comment in this mail
> thread or in issue #10963 [3]
>
> [1] https://github.com/diffplug/spotless
> [2] https://github.com/diffplug/spotless/tree/main/plugin-maven
> [3] https://github.com/apache/dolphinscheduler/issues/10963
>
> Thanks!
>
> Best Regards,
>
> Chufeng Gao
>