kwin commented on PR #41: URL: https://github.com/apache/sling-parent/pull/41#issuecomment-1857592197
> initially when switchting to this new parent which enforces the formatting to ensure existing code is complient > and later when contributing new code/PRs and build starts failing do to formatting violation Maven emits in its failure message the way to fix it: ``` [INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 3.219 s [INFO] Finished at: 2023-12-15T10:55:25+01:00 [INFO] ------------------------------------------------------------------------ [ERROR] Failed to execute goal com.diffplug.spotless:spotless-maven-plugin:2.37.0:check (default) on project ...: The following files had format violations: ... [ERROR] Run 'mvn spotless:apply' to fix these violations. [ERROR] -> [Help 1] ``` For me this looks obvious enough. Regarding IDE support, there is m2e support in spotless-maven-plugin and palantir comes with dedicated IDE plugins as well (https://github.com/palantir/palantir-java-format?tab=readme-ov-file#palantir-java-format) but the idea is really that Maven does the job and no one should need to install any additional plugins. -- 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]
