Skip to content

Update Maven wrapper#54

Merged
fabian-barney merged 2 commits into
mainfrom
codex/dependency-updates-53
May 10, 2026
Merged

Update Maven wrapper#54
fabian-barney merged 2 commits into
mainfrom
codex/dependency-updates-53

Conversation

@fabian-barney
Copy link
Copy Markdown
Owner

@fabian-barney fabian-barney commented May 10, 2026

Summary

  • update the Maven wrapper distribution from Maven 3.9.11 to Maven 3.9.15
  • use Maven 3.9.15 because the initially targeted 3.9.12 wrapper distribution URL is not available, while 3.9.15 resolves from Maven Central and is the latest stable Maven 3.x distribution found at execution time
  • keep existing Java 21 compiler release and dependency/plugin versions unchanged because versions:display-* reports them current

Validation

  • ./mvnw.cmd -B -v
  • ./mvnw.cmd -B verify -Dcentral.skipPublishing=true

Closes #53

Copilot AI review requested due to automatic review settings May 10, 2026 08:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the project’s Maven Wrapper to use Maven 3.9.12 for consistent, reproducible builds without changing the Java toolchain or any dependency/plugin versions.

Changes:

  • Bumped Maven Wrapper distributionUrl from Maven 3.9.11 to 3.9.12.
  • Left wrapper JAR (maven-wrapper 3.3.4) and the rest of the build configuration unchanged.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .mvn/wrapper/maven-wrapper.properties
@fabian-barney fabian-barney merged commit c552508 into main May 10, 2026
10 checks passed
@fabian-barney fabian-barney deleted the codex/dependency-updates-53 branch May 10, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependency update sweep

2 participants