Skip to content

fix(deps): bump spring boot and spring cloud version up #921

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

Merged
merged 7 commits into from
Jun 18, 2025

Conversation

YellowFlora
Copy link
Contributor

bump spring boot to 3.5.0
bump spring cloud to 2025.0.0

BREAKING CHANGE: it can happen that a misalignment occurs between the version of the junit-platform-launcher provided by gradle and the version of the junit-jupiter-engine provided in the spring dependencies. In this case, gradle is not able to run the tests and throws an error. It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then. That takes the version for the unit-platform-launcher from the dependency management.

bump spring boot to 3.5.0
bump spring cloud to 2025.0.0

BREAKING CHANGE: it can happen that a misalignment occurs between the version of the junit-platform-launcher provided by gradle and the version of the junit-jupiter-engine provided in the spring dependencies. In this case, gradle is not able to run the tests and throws an error. It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then. That takes the version for the unit-platform-launcher from the dependency management.
@YellowFlora YellowFlora requested a review from a team as a code owner June 17, 2025 13:22
Copy link
Contributor

github-actions bot commented Jun 17, 2025

Next major release will be 5.0.0, Show Release Notes

5.0.0 (2025-06-18)

⚠ BREAKING CHANGES

  • removed feign.jaxrs.JakartaContract from sda-commons-starter-web
    removed kafkaDltTemplate Bean from sda-commons-starter-kafka
  • deps: it can happen that a misalignment occurs between the version of the junit-platform-launcher provided by gradle and the version of the junit-jupiter-engine provided in the spring dependencies. In this case, gradle is not able to run the tests and throws an error. It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then. That takes the version for the unit-platform-launcher from the dependency management.

Features

  • remove code that was marked as deprecated and for removal (252bb2f)

Bug Fixes

  • change code that was marked as deprecated and for removal (adb2b64)
  • deps: add junit-platform-launcher in the necessary packages (354fc92)
  • deps: bump spring boot and spring cloud version up (df78ae9)

@JoergSiebahn
Copy link
Contributor

It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then.

Is it possible to ship this dependency in the required version with the module sda-commons-web-testing to make it non-breaking? Or is testRuntimeOnly a scope, we can't provide with a dependency that is used as testImplementation? Would a transitive runtimeOnly become testRuntimeOnly when using our module as testImplementation?

Copy link
Contributor

github-actions bot commented Jun 17, 2025

Test Results

  234 files  ±0    234 suites  ±0   10m 7s ⏱️ + 1m 19s
1 154 tests ±0  1 146 ✅ ±0   8 💤 ±0  0 ❌ ±0 
2 336 runs  ±0  2 316 ✅ ±0  20 💤 ±0  0 ❌ ±0 

Results for commit 304e8ec. ± Comparison against base commit 580ab9a.

♻️ This comment has been updated with latest results.

BREAKING CHANGE:
removed feign.jaxrs.JakartaContract from sda-commons-starter-web
removed kafkaDltTemplate Bean from sda-commons-starter-kafka
@YellowFlora
Copy link
Contributor Author

YellowFlora commented Jun 17, 2025

It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then.

Is it possible to ship this dependency in the required version with the module sda-commons-web-testing to make it non-breaking? Or is testRuntimeOnly a scope, we can't provide with a dependency that is used as testImplementation? Would a transitive runtimeOnly become testRuntimeOnly when using our module as testImplementation?

based of this comment from spring boot I would say it's not recommend to do this spring-projects/spring-boot#44613 (comment) but if you say it's more important to not have a breaking change I can see what I can do
Also it can be that spring-boot-starter-test is used instead of sda-commons-web-testing, then it still can be an issue

@YellowFlora
Copy link
Contributor Author

Next major release will be 5.0.0, Show Release Notes (Better Version to review)

5.0.0 (2025-06-18)

⚠ BREAKING CHANGES

  • web/starter: removed deprecated feign.jaxrs.JakartaContract
  • kafka/starter: removed deprecated kafkaDltTemplate Bean
  • test: it can happen that a misalignment occurs between the version of the junit-platform-launcher found by gradle and the version of the junit-jupiter-engine provided in the spring dependencies. In this case, gradle is not able to run the tests and throws an error. It's necessary to add testRuntimeOnly 'org.junit.platform:junit-platform-launcher' as a dependency in your build.gradle file then. That takes the version for the junit-platform-launcher from the dependency management. Adding this dependency also prepares to support Gradle 9.
  • deps: updated Spring Boot to 3.5.0, please check the release notes, if any adjustments are necessary.
  • deps: updated Spring Cloud to 2025.0.0, please check the release notes, if any adjustments are necessary.

Features

  • web/starter removed deprecated feign.jaxrs.JakartaContract (252bb2f)
  • kafka/starter removed deprecated kafkaDltTemplate Bean (252bb2f)
  • deps updated Spring Boot to 3.5.0 (df78ae9)
  • deps updated Spring Cloud to 2025.0.0 (df78ae9)

Bug Fixes

  • cleanup code that was marked as deprecated and would be removed in future versions (adb2b64)
  • test: add junit-platform-launcher in the necessary packages (354fc92)

Copy link

@YellowFlora YellowFlora merged commit 68d6159 into main Jun 18, 2025
16 of 17 checks passed
@YellowFlora YellowFlora deleted the feat/deps/update_spring branch June 18, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants