Remove standard maven-compiler plugin for applications with boot parent

What needs to be done

Provide a recipe remove-redundant-maven-compiler-plugin that removes redundant maven-compiler-plugin configurations in Spring Boot applications that have a spring-boot parent where the compiler-plugin is managed.

Why it needs to be done

Cleaner pom.xml by using Spring Boot’s plugin management

Acceptance Criteria

Given: A pom.xml with direct or indirect spring-boot-starter-parent and a standard maven-compiler-plugin configuration with only source and target When: remove-redundant-maven-compiler-plugin is applied Then: The maven-compiler-plugin is removed and Properties defining the java version are replaced with java.version property used by boot.

Additional Information

[ahmedmq]

I think i get it. Let me start and I will reach out if I get stuck

[fabapp2]

Hi @ahmedmq, thanks for your analysis!

Your observations are all correct for my understanding.

I can see the recipe doing two things:

. Always clean up properties . When no additional configuration is provided, remove the plugin

Clean up properties

• When target and source version differ, do nothing

Otherwise

• maven.compiler.source and maven.compiler.target should be retrieved from parent
• The version should be set using java.version
• Custom property for version in maven-compiler-plugin should be replaced with maven.compiler.*
  • All other usages of the custom property should to be replaced with java.version
• Fixed values for target and version should be replaced with maven.compiler.* properties set through java.version

PlatUML code: gh428.puml.txt

Removing the plugin

• After cleaning up the properties
• When the plugin has no additional configuration, remove it

It’s probably a good idea to have two Actions, one cleaning up the properties and one actually removing the plugin. This would allow us to use them separately (in two recipes) when needed.

Your thoughts on this?