Skip to content

Use JavaBestPractices in RecipeTestingBestPractices#67

Merged
timtebeek merged 3 commits intomainfrom
tim/java-best-practices
Apr 29, 2026
Merged

Use JavaBestPractices in RecipeTestingBestPractices#67
timtebeek merged 3 commits intomainfrom
tim/java-best-practices

Conversation

@timtebeek
Copy link
Copy Markdown
Member

Summary

  • Replace individual UseVarForConstructors, UseVarForPrimitive, and UseVarForTypeCast recipes in RecipeTestingBestPractices with the composite org.openrewrite.java.migrate.JavaBestPractices recipe
  • Scoped to test recipes only since JavaBestPractices includes an upgrade to Java 25

Test plan

  • Verify the YAML parses correctly and recipes resolve
  • Run mvn test to confirm no regressions

@timtebeek
Copy link
Copy Markdown
Member Author

@timtebeek timtebeek marked this pull request as draft March 27, 2026 14:27
@timtebeek timtebeek marked this pull request as ready for review April 29, 2026 22:53
@timtebeek timtebeek merged commit 31f4692 into main Apr 29, 2026
1 check passed
@timtebeek timtebeek deleted the tim/java-best-practices branch April 29, 2026 23:00
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant