-
Notifications
You must be signed in to change notification settings - Fork 5
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
Use the new/real Maven Wrapper #335
Comments
Or maybe not. After upgrading to the latest Maven, I tried running the
So, this doesn't look like it's quite ready for primetime yet. |
It might be worth giving this a try again. https://maven.apache.org/wrapper/ https://www.baeldung.com/maven-wrapper So, it looks like this might do the trick:
|
No dice. It looks like the official Maven Wrapper plugin still requires Maven 4:
|
I had the same error and have been looking for a long time for a solution. Finally, I found it. When you just call the goal with If you call the goal explicitly with the latest plugin version, it works: mvn org.apache.maven.plugins:maven-wrapper-plugin:3.1.1:wrapper |
Awesome! Thanks for letting me know. I'll give this a try when I start preparing for the Winter offering of my course. |
You're welcome! I imagine providing a controlled environment is especially useful in the context of Java courses. It's definitely a blessing for me who was otherwise stuck on a not-so-recent Maven version from the Ubuntu LTS repository which doesn't work with the latest Java versions. |
Well, the Gilded Rose kata uses the Apache Maven Wrapper: emilybache/GildedRose-Refactoring-Kata@0eab197 So, we should be able to, also. |
…wrapper Use the official Maven Wrapper #335
It looks like Maven wrapper has been promoted to be a first-class citizen in Maven:
https://maven.apache.org/plugins/maven-wrapper-plugin/index.html
I should recreate the Maven wrapper scripts and jars using that plugin.
The text was updated successfully, but these errors were encountered: