gnodet opened a new pull request, #12165: URL: https://github.com/apache/maven/pull/12165
## Summary - When a remote parent POM declares a plugin with an explicit version in `<build><plugins>` (not via `<pluginManagement>`), `mvnup`'s plugin upgrade strategy now correctly adds the override in `<build><plugins>` instead of only `<pluginManagement>` - The analysis compares each plugin's effective version in `build/plugins` against `build/pluginManagement/plugins` to determine upfront whether a `<pluginManagement>` override alone is sufficient or a direct `<plugins>` entry is needed - Unlike the previous approach (#12157), this avoids adding unnecessary `<plugins>` entries when the version is controlled via `<pluginManagement>` Supersedes #12157 and #12159. ## Test plan - [x] New test `shouldNotAddDirectOverrideWhenVersionFromPluginManagement` verifies no false-positive `<plugins>` entries for `org.apache:apache:23` (version comes from PM) - [x] New test `shouldNotDuplicatePluginInBuildPluginsWhenAlreadyDeclared` verifies no duplicates when plugin is already locally declared - [x] All 29 existing + new PluginUpgradeStrategy tests pass 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
