Skip to content

Updated dependencies: maven plugins and library deps - #1242

Merged
johnmay merged 10 commits into
mainfrom
deps/updates/202511
Nov 9, 2025
Merged

Updated dependencies: maven plugins and library deps#1242
johnmay merged 10 commits into
mainfrom
deps/updates/202511

Conversation

@egonw

@egonw egonw commented Nov 9, 2025

Copy link
Copy Markdown
Member
  • centralized versions to solve inconsistent version dependencies
  • updated log4j
  • version Maven plugin updates

@egonw
egonw requested a review from johnmay November 9, 2025 09:01
@egonw
egonw removed the request for review from johnmay November 9, 2025 09:12
@egonw
egonw marked this pull request as draft November 9, 2025 09:12
@johnmay

johnmay commented Nov 9, 2025

Copy link
Copy Markdown
Member

Rather than use the properties I think the correct way to do this is to declare the plugin in the parent with the version then omit the version in the child i.e. we should use <dependnecyManagment> for log4j and <pluginManagement> for jacoco etc. The property defs are still sometimes useful

@egonw

egonw commented Nov 9, 2025

Copy link
Copy Markdown
Member Author

we should use

okay, I will do that next: #1244

For now, please merge in this one.

@egonw
egonw marked this pull request as ready for review November 9, 2025 10:33
@egonw
egonw requested a review from johnmay November 9, 2025 10:33
@sonarqubecloud

sonarqubecloud Bot commented Nov 9, 2025

Copy link
Copy Markdown

@egonw egonw linked an issue Nov 9, 2025 that may be closed by this pull request
@johnmay
johnmay merged commit ae0d100 into main Nov 9, 2025
12 checks passed
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.

plugin dependency version inconsistensies

2 participants