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
feat(maven-wrapper): Add manager to update maven wrapper version #15660
feat(maven-wrapper): Add manager to update maven wrapper version #15660
Conversation
Co-authored-by: Rhys Arkins <rhys@arkins.net>
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.
needs tests
lib/modules/manager/maven-wrapper/__fixtures__/only-maven/maven-wrapper.properties
Outdated
Show resolved
Hide resolved
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
@benkelaar and me created this demo repo, which demonstrates updating maven wrapper and maven. This was run locally with Any other scenario's you want us to map out? |
The example repo looks good to me |
I'm still really hoping to see this merged 🤞 Thank you all for this great work and your continued efforts! |
some small issues need to be fixed before merge |
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
🎉 This PR is included in version 34.111.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Merge request for Issue #3566 , we still need to add some unit tests to get 100% coverage and look at whether the documentation needs to be updated, but we would like some first feedback on whether we've taken the right approach.
Changes
Add custom manager to do maven wrapper version upgrades of both maven version and maven wrapper version.
Context
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: