Skip to content

Conversation

timtebeek
Copy link
Member

Closes #75

@timtebeek
Copy link
Member Author

Wasn't sure about:

  • naming of the recipe
  • if we should also include gradle, or future proof naming/description
  • whether to include a test

While I'm not sure how much Wro4j is still used out in the wild, this should at the very least make it easier to migrate the Spring PetClinic 1.5.x as an example.

@jkschneider
Copy link
Member

if we should also include gradle

Gradle support is going to keep improving so we can come back to that.

or future proof naming/description

I think it's ok. The recipe could switch over multiple versions if that time ever comes without changing the name.

@jkschneider jkschneider merged commit a9ca8a7 into openrewrite:main Apr 1, 2022
@timtebeek timtebeek deleted the UpgradeWro4jMavenPluginVersion branch April 2, 2022 06:19
@tkvangorder tkvangorder added the enhancement New feature or request label Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Provide recipe to upgrade wro4j to a version that is compatible with Java 11
3 participants