Skip to content

Update spring boot version 4.0.1 - #275

Closed
dhammernik wants to merge 2 commits into
xenit-eu:mainfrom
dhammernik:update-spring-boot-4.0.1
Closed

Update spring boot version 4.0.1#275
dhammernik wants to merge 2 commits into
xenit-eu:mainfrom
dhammernik:update-spring-boot-4.0.1

Conversation

@dhammernik

Copy link
Copy Markdown

No description provided.

renovate Bot and others added 2 commits February 2, 2026 10:02
(cherry picked from commit fdd44ae)
… for alignment with latest conventions. Updated tests accordingly and added necessary dependencies in build files.

(cherry picked from commit 621d868669ad1e6744912a6c0abdc3ab1d5da830)
@dhammernik
dhammernik requested a review from a team as a code owner February 2, 2026 09:13
@sonarqubecloud

sonarqubecloud Bot commented Feb 2, 2026

Copy link
Copy Markdown

internalPlatform platform(project(':thunx-dependencies'))

compileOnly 'org.projectlombok:lombok'
compileOnly 'org.jspecify:jspecify'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This compileOnly dependency that is added everywhere is technically not needed for the spring boot 4 update.

api platform("org.springframework.cloud:spring-cloud-dependencies:2025.0.0")
api platform("org.springframework.boot:spring-boot-dependencies:4.0.1")
api platform("org.springframework.cloud:spring-cloud-dependencies:2025.1.0")
api platform("org.testcontainers:testcontainers-bom:1.20.4")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line overwrites the testcontainers version set by spring-boot. testcontainers was updated to v2 in spring-boot 4. I think it should be relatively straightforward to remove this line and update the testcontainers v2 dependencies in all the build.gradle files.

@thijslemmens

Copy link
Copy Markdown

@dhammernik for your contribution. Your work has been used in this PR that has been merged:
#302

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.

3 participants