Skip to content

Updated dependencies: maven plugins and library deps#1242

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

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

Conversation

@egonw
Copy link
Copy Markdown
Member

@egonw egonw commented Nov 9, 2025

  • 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
Copy link
Copy Markdown
Member

johnmay commented Nov 9, 2025

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
Copy link
Copy Markdown
Member Author

egonw commented Nov 9, 2025

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
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 9, 2025

@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