-
Notifications
You must be signed in to change notification settings - Fork 42
Upgrade maven-enforcer-plugin 3.1.0 -> 3.2.1 #487
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
Upgrade maven-enforcer-plugin 3.1.0 -> 3.2.1 #487
Conversation
Suggested commit message:
|
Looks good. No mutations were possible for these changes. |
2 similar comments
Looks good. No mutations were possible for these changes. |
Looks good. No mutations were possible for these changes. |
44eb1e9
to
2352580
Compare
Looks good. No mutations were possible for these changes. |
3 similar comments
Looks good. No mutations were possible for these changes. |
Looks good. No mutations were possible for these changes. |
Looks good. No mutations were possible for these changes. |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. ⚠ Warning: custom changes will be lost. |
f1673a6
to
238f3e0
Compare
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.
We can now enable other checks, but will do that in a separate PR.
Looks good. No mutations were possible for these changes. |
Suggested commit message:
|
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.
LGTM, added a suggested commit message. Didn't dive in too deep into the new things, but curious about the following PRs 😄.
@@ -913,7 +918,6 @@ | |||
<version>${version.maven}</version> | |||
</requireMavenVersion> | |||
<requirePluginVersions /> | |||
<requireSameVersionsReactor /> |
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.
It sounds similar to the ReactorModuleConvergence
.
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.
Yep. Unfortunately ReactorModuleConvergence
is not compatible with -pl
, so not something we can enable.
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.
Tweaked the suggested commit message :)
@@ -913,7 +918,6 @@ | |||
<version>${version.maven}</version> | |||
</requireMavenVersion> | |||
<requirePluginVersions /> | |||
<requireSameVersionsReactor /> |
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.
Yep. Unfortunately ReactorModuleConvergence
is not compatible with -pl
, so not something we can enable.
I filed #489 :) |
This PR contains the following updates:
3.1.0
->3.2.1