-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add recipe for Spring Boot 3.4 migration #629
Conversation
...tWithSpringBoot_3_4/java/org/openrewrite/java/spring/boot3/SpringBootVersionUpgradeTest.java
Outdated
Show resolved
Hide resolved
…ot3/SpringBootVersionUpgradeTest.java Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
...tWithSpringBoot_3_4/java/org/openrewrite/java/spring/boot3/SpringBootVersionUpgradeTest.java
Outdated
Show resolved
Hide resolved
...tWithSpringBoot_3_4/java/org/openrewrite/java/spring/boot3/SpringBootVersionUpgradeTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick work here!
When will this recipe be available via maven central? I would like to use it. |
hi! Right now this recipe doesn't do anything just yet; it merely bumps the dependencies. We're looking to include the steps needed for Spring Framework 6.2 ( #627 ) as well as changes needed for Spring Boot 3.4 before we recommend running that migration through OpenRewrite. We wouldn't want you to have a bad experience with a partial recipe. Help always welcome to fill in the gaps with regards to recipe coverage. |
@timtebeek thanks for the reply! Just saw that commit :) |
* Add recipe for springboot 3.4 migration * Add test * Fix comment * Fix test * Update src/testWithSpringBoot_3_4/java/org/openrewrite/java/spring/boot3/SpringBootVersionUpgradeTest.java Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Drop expected cycles that made changes * Expect 3.3 before 3.4 is released --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Tim te Beek <[email protected]>
What's changed?
Two changes
What's your motivation?
Anything in particular you'd like reviewers to focus on?
org.openrewrite.java.spring.boot3.SpringBoot3BestPractices
recipe, as was the case with the 3.3 upgrade. However, since the recipe is now extracted/standalone do we still want this to be the case?Anyone you would like to review specifically?
@Laurens-W @timtebeek
Any additional context
Checklist