kezhenxu94 commented on PR #11272: URL: https://github.com/apache/dolphinscheduler/pull/11272#issuecomment-1205376361
> For developer's convenience, I suggest setting `Spotless` executions configuration as follows: > > > > ```xml > > <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 personally used pre-commit hook in my own project and I prefer to use this. Also even we have the hook we should still keep the style check in CI because that can be skipped in developers' local machine. -- 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]
