-
Notifications
You must be signed in to change notification settings - Fork 267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
includeOnlyReactor: Update versions of only internal dependencies reachable from root in multi-module/reactor build #10
Comments
I also need such a feature. Please consider implementing it. |
What about offering a PR ? |
It sounds you should use |
@khmarbaise, this is not the described use case. I believe Now, not all dependencies have to be updated.
We need to update only those in "category (A)". For example:
Again, it is exactly the opposite to what Note that, in general, all those modules in example ( One more point. Even though a module is referenced in |
Ok..misunderstanding on my site.. |
I'd like to note that the suggested |
This issue originates (copied & pasted) from this stackoverflow question - it might be a valuable feature for this plugin.
Problem
When I run the following command for the
root
project, it checks for updates of ALL dependencies:I want to limit versions update for only those dependencies which are reachable from the
root
:The group/artifact ids of these dependencies are in the current reactor build (multi-module build).
Any component with group and artifact ids which are not part of the build.
For example, I don not want to update external dependencies like
junit
orcommons-logging
.There is an option
excludeReactor
. And I need an opposite likeincludeOnlyReactor
.Otherwise, it is unreliable and tedious work to specify all possible artifact patterns "managed" within sources being built via
includes
option.The problem is that changes in the internal dependencies are are controllable. External updates are not.
Examples
For example, the following automatic update of external dependency just gets through unnoticed:
See also more clarifications in the comment below.
The text was updated successfully, but these errors were encountered: