Issue #101: run bestPractices recipe with build

Author: Anmol202005

.github/workflows/comment-pr.yml

@@ -20,8 +20,8 @@
         <maven.compiler.plugin>3.11.0</maven.compiler.plugin>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 
-        <rewrite.maven.plugin>6.16.0</rewrite.maven.plugin>
-        <recipe.bom.version>3.13.0</recipe.bom.version>
+        <rewrite.maven.plugin>6.17.0</rewrite.maven.plugin>
+        <recipe.bom.version>3.14.0</recipe.bom.version>
         <junit.version>5.13.4</junit.version>
         <assertj.version>3.27.4</assertj.version>
 
@@ -173,6 +173,9 @@
                 <version>${rewrite.maven.plugin}</version>
                 <configuration>
                     <activeRecipes>
+                        <recipe>org.checkstyle.recipes.OpenRewriteRecipeBestPractices</recipe>
+                        <recipe>org.checkstyle.recipes.JavaRecipeBestPractices</recipe>
+                        <recipe>org.checkstyle.recipes.RecipeTestingBestPracticesSubset</recipe>
                         <recipe>CheckstyleAutoFixConfigured</recipe>
                     </activeRecipes>
                     <exclusions>
@@ -188,7 +191,7 @@
                     <dependency>
                         <groupId>org.openrewrite.recipe</groupId>
                         <artifactId>rewrite-rewrite</artifactId>
-                        <version>RELEASE</version>
+                        <version>0.12.1</version>
                     </dependency>
                 </dependencies>
                 <executions>

.github/workflows/receive-pr.yml

@@ -44,13 +44,14 @@ preconditions:
 recipeList:
   - org.openrewrite.java.recipes.BlankLinesAroundFieldsWithAnnotations
   - org.openrewrite.java.recipes.CorrectlySpacedDescriptions
-#  - org.openrewrite.java.recipes.ExecutionContextParameterName
+  - org.openrewrite.java.recipes.ExecutionContextParameterName:
+      parameterName: executionContext
   - org.openrewrite.java.recipes.IsLiteralNullRecipe
   - org.openrewrite.java.recipes.MissingOptionExample
   - org.openrewrite.java.recipes.NoMutableStaticFieldsInRecipes
   - org.openrewrite.java.recipes.RecipeEqualsAndHashCodeCallSuper
   - org.openrewrite.java.recipes.UseTreeRandomId
-  - org.openrewrite.java.recipes.migrate.RemoveTraitsUsageRecipes
+  - org.openrewrite.java.InlineMethodCalls
   - org.openrewrite.staticanalysis.NeedBraces
   - org.openrewrite.staticanalysis.RemoveSystemOutPrintln
   - org.openrewrite.java.RemoveAnnotation:

pom.xml

@@ -60,7 +60,8 @@ private final class RemoveRedundantImportsVisitor extends JavaIsoVisitor<Executi
         private Path sourcePath;
 
         @Override
-        public J.CompilationUnit visitCompilationUnit(J.CompilationUnit cu, ExecutionContext ctx) {
+        public J.CompilationUnit visitCompilationUnit(J.CompilationUnit cu,
+                                                      ExecutionContext executionContext) {
 
             this.sourcePath = cu.getSourcePath().toAbsolutePath();