-
-
Couldn't load subscription status.
- Fork 78
👷 Introduce Maven Wrapper #144
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
Conversation
With the Maven Wrapper the contributor does not need to install Maven first. It makes sure that a build is reproducible by using always the same version of Maven for the build.
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.
As Travis is malfunctioning (unless you pay for it) we scrapped it, so please redo this PR to fix merge conflicts if you can.
IMHO the wrapper is not needed. Developers of the library knows perfectly how to use the Maven. Users may add it into their projects if they wish.
In the same time it adds more files to source tree that also should be updated, cloned, etc.
Uh oh!
There was an error while loading. Please reload this page.
With the Maven Wrapper the contributor does not need to install Maven first. It makes sure that a build is reproducible by using always the same version of Maven for the build.
This change is Reviewable