Skip to content

Spring Boot 3 review#17

Merged
PhilWakelin merged 14 commits intomainfrom
piw-review
Feb 9, 2026
Merged

Spring Boot 3 review#17
PhilWakelin merged 14 commits intomainfrom
piw-review

Conversation

@PhilWakelin
Copy link
Member

Update prereqs to CICS TS 6.1
Fix build.gradle
Remove duplication
Update MAINTAINERS

Copy link
Contributor

@IvanHargreaves IvanHargreaves left a comment

Choose a reason for hiding this comment

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

Mostly all good changes. I should have spotted most of these, I guess I was focused on what changed, not what hadn't changed that should!

The one change we do want to retain is the bootWAR false though. That's a good thing, we have no use for the larger executable, just use the plain WAR instead.

@PhilWakelin PhilWakelin requested a review from Manvi-55 February 9, 2026 09:34
@PhilWakelin
Copy link
Member Author

The one change we do want to retain is the bootWAR false though. That's a good thing, we have no use for the larger executable, just use the plain WAR instead.

PR updated with instructions to use the gradle build task to avoid bootWar build, and with Maven build spring-boot.repackage.skip property.

Copy link
Contributor

@IvanHargreaves IvanHargreaves left a comment

Choose a reason for hiding this comment

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

Something I should have spotted before, even though Spring Boot 3 works with the Jakarta EE 9 libraries, I think the official base-line is Java 17 and Jakarta EE 10.

So we'll probably need to put Jakarta EE 10 and Servlet-6.0 in the pre-reqs and sample.

@PhilWakelin PhilWakelin merged commit 0684316 into main Feb 9, 2026
5 checks passed
@PhilWakelin PhilWakelin deleted the piw-review branch February 9, 2026 15:11
@PhilWakelin PhilWakelin restored the piw-review branch February 9, 2026 15:32
@PhilWakelin PhilWakelin deleted the piw-review branch February 10, 2026 09:31
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

Comments