Skip to content
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

Bump org.apache.maven.plugins.maven-wrapper-plugin from 3.1.1 to 3.2.0 #8

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

c00ler
Copy link
Member

@c00ler c00ler commented Mar 12, 2023

No description provided.

@c00ler c00ler added the dependencies Dependency updates label Mar 12, 2023
@c00ler c00ler self-assigned this Mar 12, 2023
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-wrapper-plugin</artifactId>
<version>3.2.0</version>
</plugin>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need this plugin now?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not a plugin definition, it's a <pluginManagement>. We are overriding the version shipped with the maven and can use mvn wrapper:wrapper in the future. Otherwise, we would need always to use full plugin coordinates.

For example in https://github.com/gradle/maven-build-scan-quickstart dependabot made the changes by specifying the full coordinates in the command line. If now one will execute mvn wrapper:wrapper -Dmaven=3.9.0 using the latest maven 3.9.0, the wrapper version will be downgraded back to 3.1.1. See: https://scans.gradle.com/s/il4shfvrnbz7e/console-log?page=1#L8

@c00ler c00ler merged commit 88109d9 into main Mar 13, 2023
@c00ler c00ler deleted the avenderov/update-maven-wrapper branch March 13, 2023 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependency updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants