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

Add Maven 4 #212

Merged
merged 1 commit into from
Nov 18, 2022
Merged

Add Maven 4 #212

merged 1 commit into from
Nov 18, 2022

Conversation

dmikusa
Copy link
Contributor

@dmikusa dmikusa commented Nov 17, 2022

Summary

This PR adds Maven 4 (alpha). This is added as a second major branch for anyone that is interested in trying it.

It also adds the ability to select the major version of Maven that is used. The default version is set to version 3.x. By setting BP_MAVEN_VERSION, you can override this.

This also includes a minor fix where the logger was not being plumbed through to the Maven Manager object. That could have resulted in loss of some log events.

While this adds everything to support Maven 4, the update job for Maven 4 isn't going to work until this pipeline-builder request is merged. It does not require a new pipeline-builder version, as the changes required are only in the maven dependency action and that is not versioned.

Resolved #204

Use Cases

Try using Maven 4.

This commit adds Maven 4 (alpha). This is added as a second major branch for anyone that is interested in trying it.

It also adds the ability to select the major version of Maven that is used. The default version is set to version 3.x. By setting `BP_MAVEN_VERSION`, you can override this.

This also includes a minor fix where the logger was not being plumbed through to the Maven Manager object. That could have resulted in loss of some log events.

Signed-off-by: Daniel Mikusa <dan@mikusa.com>
@dmikusa dmikusa added semver:minor A change requiring a minor version bump type:enhancement A general enhancement labels Nov 17, 2022
@dmikusa dmikusa requested a review from a team November 17, 2022 04:28
@dmikusa dmikusa merged commit aae8b22 into main Nov 18, 2022
@dmikusa dmikusa deleted the maven-4 branch November 18, 2022 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:minor A change requiring a minor version bump type:enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Maven 4
2 participants