-
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
#499 - Let update-parent support -DallowMajorUpdates and alike #500
#499 - Let update-parent support -DallowMajorUpdates and alike #500
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the PR - can you come up with integration tests as well for the new functionality?
please also rebase/merge with current develop branch to resolve the conflicts.
@askoog thanks for refresh PR. as @stefanseifert said, IT test will be appreciated |
@askoog Hi there, do you still want to finish this PR? If you want, I could take it over and finish what's left to do. Please reach out. |
@ajarmoniuk Please feel free if you want to continue with this PR. I tried to fix errors and also added some tests, but when I rebased on master I see that you have made quite a few changes in the latest version and some of the methods I used have been removed :/ |
Actually we'd like to give this allow...Updates system a slight overhaul, at least in its inner workings, since those options are not mutually independent and it's much better to represent them as e.g. an enum, at least internally. And, obviously, furnishing this mojo with this feature is coming :) |
I see. Yes that sounds like a good idea. I guess it will be easier to implement the functionality in other mojos as well then. The allowXxxUpdates is a great addition to this plugin and we use it very frequently. We have several hundred maven projects that we continuously keep up to date using this plugin and internally we have different major versions for our development platform projects based on java version, spring version etc. Upgrading to a different major is often a bigger task than just upgrading patch versions and therefore it is great to be able to perform automatic updates for just pathches/minors and not majors. Anyhow, I will leave this PR to you, if you want to close it, go ahead. Feel free to reach out if there is anything I can help with |
Working on it. I'll wait till #733 is merged since I'd like to use the Incrementing a segment to obtain the upper bound brings caveats with itself -- we may indadvertedly update to a beta or a snapshot version from the next major release because these lie before the .0 versions. Which my PR #733 fixes. |
Implement support for
allowMajorUpdates
,allowMinorUpdates
andallowIncrementalUpdates
in update-parent