ruanwenjun commented on PR #11272: URL: https://github.com/apache/dolphinscheduler/pull/11272#issuecomment-1205383986
> For developer's convenience, I suggest setting `Spotless` executions configuration as follows: > > ``` > <executions> > <execution> > <goals> > <goal>apply</goal> > </goals> > <phase>compile</phase> > </execution> > </executions> > ``` > > In this way, `Spotless` will automatically fix the style / formatting issues when compiling the project. However, we need to keep `Checkstyle` so that CI could make sure the style is correct in case developers do not compile and test the code before pushing it to remote repo. > > Another way is to use `git pre-commit hook` and remove `Checkstyle`. WDYT @kezhenxu94 @ruanwenjun I agree with removing `Checkstyle` in ci, if we keep the `Checkstyle` we may need to maintain the style is the same in `Spotless` and `Checkstyle`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
