You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I had searched in the issues and found no similar feature requirement.
Description
Currently, DS CI checks formatting errors. However, it will be much more conveniently if we could find those style errors and automatically fix them before pushing the code.
We could introduce spotless plugin into DS and use it to automatically fix the style.
At present, DS uses checkstyle plugin. However, there are tons of legacy formatting errors in the project 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 over CheckStyle is in addition to check the formatting of the code it also has apply goal that fixes all the style and formatting. 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.
Search before asking
Description
spotlessplugin into DS and use it to automatically fix the style.Use case
checkstyleplugin. However, there are tons of legacy formatting errors in the project every time we build the project. Thus, in practice, some developers tend to use-Dcheckstyle.skipflag when building the project and this make thestyle checka little bit awkward.SpotlessoverCheckStyleis in addition to check the formatting of the code it also hasapplygoal that fixes all the style and formatting. WithSpotless, we could fix the legacy formatting errors once for all and developers could fix new format errors with a single line of commandmvn spotless:apply.Related issues
No response
Are you willing to submit a PR?
Code of Conduct