Skip to content

fix: update Maven wrapper to 3.9.14#1761

Merged
gnodet merged 1 commit into
masterfrom
fix/maven-wrapper-version
Apr 7, 2026
Merged

fix: update Maven wrapper to 3.9.14#1761
gnodet merged 1 commit into
masterfrom
fix/maven-wrapper-version

Conversation

@gnodet
Copy link
Copy Markdown
Member

@gnodet gnodet commented Apr 3, 2026

Summary

  • Update Maven wrapper distribution URL from 4.0.0-rc-4 to 3.9.14
  • The build was reverted to Maven 3.9.x compatibility in fix: Revert build to Maven 3.9.x compatibility #1699, but the wrapper properties still referenced Maven 4.0.0-rc-4
  • Verified nisse versioning works correctly with Maven 3.9.x

Summary by CodeRabbit

  • Chores
    • Updated Maven Wrapper to use Apache Maven 3.9.14, reverting from the 4.0.0-rc-4 prerelease binary.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 054a7d6e-e709-42c4-a14d-7397e4f96295

📥 Commits

Reviewing files that changed from the base of the PR and between b566795 and 1fe4b1b.

📒 Files selected for processing (1)
  • .mvn/wrapper/maven-wrapper.properties
🚧 Files skipped from review as they are similar to previous changes (1)
  • .mvn/wrapper/maven-wrapper.properties

📝 Walkthrough

Walkthrough

The Maven Wrapper configuration was updated to change the distributionUrl from apache-maven-4.0.0-rc-4-bin.zip to apache-maven-3.9.14-bin.zip, altering the Maven binary downloaded by the wrapper.

Changes

Cohort / File(s) Summary
Maven Wrapper Configuration
\.mvn/wrapper/maven-wrapper.properties
Updated distributionUrl value: replaced apache-maven-4.0.0-rc-4-bin.zip with apache-maven-3.9.14-bin.zip (other wrapper properties unchanged).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I swapped a future RC for trusty three-nine-fourteen,
A tiny hop, a lighter scene.
Wrapper fetches steady ground,
No more release-candidate bound.
Hooray for builds that stay serene! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title states the Maven wrapper is updated to 3.9.14, but the PR description and objectives indicate the update should be to 3.9.9, creating a version mismatch. Update the title to 'fix: update Maven wrapper to 3.9.9' to align with the actual version change and PR objectives.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/maven-wrapper-version

Comment @coderabbitai help to get the list of available commands and usage tips.

@gnodet gnodet force-pushed the fix/maven-wrapper-version branch from b566795 to 1fe4b1b Compare April 7, 2026 02:00
@gnodet gnodet changed the title fix: update Maven wrapper to 3.9.9 fix: update Maven wrapper to 3.9.14 Apr 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 7, 2026

@gnodet gnodet merged commit 1301c8d into master Apr 7, 2026
11 checks passed
@gnodet gnodet added the build label Apr 7, 2026
@gnodet gnodet added this to the 4.0.11 milestone Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant