Skip to content

Consolidate Spring Data migration YAML into two files#986

Merged
timtebeek merged 3 commits intomainfrom
tim/consolidate-spring-data
Apr 2, 2026
Merged

Consolidate Spring Data migration YAML into two files#986
timtebeek merged 3 commits intomainfrom
tim/consolidate-spring-data

Conversation

@timtebeek
Copy link
Copy Markdown
Member

Summary

  • Merges five individual Spring Data version migration files (2.3, 2.5, 2.7, 3.0, 3.4) into two: spring-data-2.yml and spring-data-3.yml
  • Inlines intermediate recipe steps (UseJpaRepositoryGetById, UseJpaRepositoryDeleteAllInBatch, UseJpaRepositoryGetReferenceById) directly into their parent upgrade recipes
  • Chains each version upgrade to include its predecessor (2.5 → 2.3, 2.7 → 2.5)
  • All recipe names are preserved so external references from spring-boot and spring-framework YAML files remain valid

Test plan

  • ./gradlew test --tests "org.openrewrite.java.spring.data.*" passes

Merge five individual spring-data version migration files (2.3, 2.5, 2.7,
3.0, 3.4) into two: spring-data-2.yml and spring-data-3.yml. Inline
intermediate recipe steps directly into parent upgrade recipes and chain
each version upgrade to include its predecessor. All recipe names are
preserved so external references remain valid.
@timtebeek
Copy link
Copy Markdown
Member Author

@steve-aom-elliott your changes in #984 are what prompted me to have a closer look, but I didn't want to complicate your PR; isolated here it's easier to see and merge I think. Feel free to immediately merge if you agree.

@timtebeek timtebeek merged commit 1dc43ea into main Apr 2, 2026
1 check passed
@timtebeek timtebeek deleted the tim/consolidate-spring-data branch April 2, 2026 06:24
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Apr 2, 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.

2 participants