-
Notifications
You must be signed in to change notification settings - Fork 206
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
[refactoring] converting petclinic to Spring Boot 3 results in broken project #864
Comments
I thought Rewrite wanted to get this fixed... probably still isn't |
Here are the things that the Spring Boot Migrator project includes at the moment with regards to converting projects to use Spring Boot 3: https:/spring-projects-experimental/spring-boot-migrator/blob/main/components/sbm-recipes-boot-upgrade/src/main/resources/recipes/boot-2.7-3.0-dependency-version-update.yaml So it looks to me like we should have our own, custom-made Spring Tools 4 recipe that contains the things from rewrite-spring (that we use at the moment) and pick additional open-rewrite-based things from the SBM yaml mentioned above, and use that (for the moment) to offer users the ability to convert Spring Boot projects to Spring Boot 3. Over time, those things should converge - and moved back to rewrite-spring - but in the short-term, having a custom recipe to aggregate the pieces makes some sense to me. |
Should be fixed once this is merged: openrewrite/rewrite-migrate-java#144 |
@martinlippert shall we perhaps switch to rewrite snapshots for now? Less stability I suppose but always on the latest Rewrite code with the latest fixes and additions? |
@BoykoAlex Yes, we should switch to snapshot versions. In addition to that I am thinking the situation where a new Spring Boot (patch) version is being published. It seems to me like the recipe needs to always update to the latest Spring Boot 3.0.x version (instead of having a fixed one in the recipe). But we should probably have that in a separate issue. |
Should be fixed. Rewrite is fixed and rewrite snapshots consumed by sts4. |
Converting the standard petclinic sample application to Spring Boot 3 results in a broken project:
the jakarta persistence api dependency gets added, but this doesn't seem to be necessary
this dependency doesn't get added:
so that compile errors are the result.
The text was updated successfully, but these errors were encountered: