Skip to content
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

Merged
merged 7 commits into from
Nov 18, 2024
Merged

Conversation

nielsdebruin
Copy link
Contributor

@nielsdebruin nielsdebruin commented Nov 18, 2024

What's changed?

Two changes

  • Add a recipe for updating to Spring Boot 3.4.
  • The Spring Boot 3 best practices recipe has been extracted from the Spring Boot 3.3 update recipe.

What's your motivation?

Anything in particular you'd like reviewers to focus on?

  • We don't have a recipe for Spring 6.2 yet, which I think should be part of the update.
  • I have made the 3.4 upgrade part of the 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

  • I've added unit tests to cover both positive and negative cases
  • I've read and applied the recipe conventions and best practices
  • I've used the IntelliJ IDEA auto-formatter on affected files

@nielsdebruin nielsdebruin added the recipe Recipe requested label Nov 18, 2024
@nielsdebruin nielsdebruin self-assigned this Nov 18, 2024
…ot3/SpringBootVersionUpgradeTest.java

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@timtebeek timtebeek changed the title Add recipe for springboot 3.4 migration Add recipe for Spring Boot 3.4 migration Nov 18, 2024
@timtebeek timtebeek marked this pull request as ready for review November 18, 2024 12:57
Copy link
Contributor

@timtebeek timtebeek left a 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!

@timtebeek timtebeek merged commit 5a9d9c6 into main Nov 18, 2024
2 checks passed
@timtebeek timtebeek deleted the recipe/springboot-34 branch November 18, 2024 13:15
@zyberzebra
Copy link

When will this recipe be available via maven central? I would like to use it.

@timtebeek
Copy link
Contributor

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.

@zyberzebra
Copy link

@timtebeek thanks for the reply! Just saw that commit :)

sambsnyd pushed a commit that referenced this pull request Jan 22, 2025
* 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
recipe Recipe requested
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants