Skip to content

Update springboot 4#234

Open
newgro wants to merge 5 commits into
MarcGiffing:masterfrom
newgro:update-springboot-4
Open

Update springboot 4#234
newgro wants to merge 5 commits into
MarcGiffing:masterfrom
newgro:update-springboot-4

Conversation

@newgro
Copy link
Copy Markdown

@newgro newgro commented Apr 30, 2026

I was updating the libraries for springboot 4.0.6

The new modularized inclusion of external systems is applied.
I added some commits for code cleanup and accessibility improvements.

To simplify library update we need to cleanup the code.
Otherwise we can not really see deprecations.

Removed unused imports.
Removed unnecessary public modifiers from spring-bean definitions.
Added some ignore flags to hide false positive warnings.
@newgro newgro force-pushed the update-springboot-4 branch 2 times, most recently from e3d5652 to dc278c4 Compare April 30, 2026 11:12
Comment thread pom.xml Outdated
<apache-shiro.version>2.0.5</apache-shiro.version>
<wicket.version>10.6.0</wicket.version>
<wicketstuff.version>10.6.0</wicketstuff.version>
<java.version>21</java.version>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why not 17?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I didn’t put much thought into that choice — it’s just the base JDK version used across my libraries. Using 17 would work just as well.

newgro added 4 commits May 5, 2026 08:58
If the tab was already close message sending failed on null
connections.
Improves accessibility (screen readers)
Is relevant for SEO
Is recommended as part of HTML5 compliance
@newgro newgro force-pushed the update-springboot-4 branch from dc278c4 to 5921815 Compare May 5, 2026 07:03
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