Set Lombok to provided scope

[yeikel]

Description

Lombok should not be present in the classpath because it is only used at compile time. As per the docs, the dependency should set to the provided scope.

Describe the situation before applying the recipe

            <groupId>org.projectlombok</groupId>
            <artifactId>lombok</artifactId>
            <version>${lombok.version}</version>

Describe the situation after applying the recipe

diff --git a/pom.xml b/pom.xml
index 9d43d17f8..795236488 100644
--- a/pom.xml
+++ b/pom.xml
@@ -245,6 +245,7 @@
             <groupId>org.projectlombok</groupId>
             <artifactId>lombok</artifactId>
             <version>${lombok.version}</version>
+            <scope>provided</scope>
         </dependency>

Any additional context

This can be implemented as a YAML recipe using the change scope recipe

Official docs: https://projectlombok.org/setup/maven

[timtebeek]

That's a good suggestion indeed, thanks! We have a couple other Lombok recipes in rewrite-migrate-java; I'll move this there and we can add it to a Lombok best practices recipe that includes the others too.