fix: update Maven wrapper to Maven 3.9.12 and use it in CI#3281
fix: update Maven wrapper to Maven 3.9.12 and use it in CI#3281csviri merged 1 commit intooperator-framework:mainfrom
Conversation
xstefank
commented
Apr 8, 2026
xstefank
commented
Signed-off-by: xstefank <xstefank122@gmail.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the Maven wrapper infrastructure to Maven 3.9.12 and modernizes the wrapper scripts from version 3.1.0 to 3.3.4. The key architectural change is moving from a JAR-based downloader to script-based Maven distribution downloading. The CI workflows are updated to consistently use the wrapper (./mvnw) instead of relying on pre-installed Maven.
Changes:
- Updated Maven version from 3.8.4 to 3.9.12 in wrapper configuration
- Rewrote mvnw shell script with improved error handling, platform detection, and download capabilities
- Rewrote mvnw.cmd as a hybrid batch/PowerShell script for better Windows compatibility
- Removed Java-based MavenWrapperDownloader.java (now handled by scripts)
- Updated maven-wrapper.properties to reference Maven 3.9.12 and added metadata fields
- Updated 4 CI workflow files to use
./mvnwinstead of directmvncommands
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| mvnw | Complete rewrite with improved error handling, platform detection, and SHA-256 validation support |
| mvnw.cmd | Hybrid batch/PowerShell implementation with direct Maven distribution downloading |
| .mvn/wrapper/maven-wrapper.properties | Updated to Maven 3.9.12, simplified format, added wrapper metadata |
| .mvn/wrapper/MavenWrapperDownloader.java | Removed (functionality now in scripts) |
| .github/workflows/snapshot-releases.yml | Updated to use ./mvnw wrapper |
| .github/workflows/release-project-in-dir.yml | Updated to use ./mvnw wrapper |
| .github/workflows/e2e-test.yml | Updated to use ./mvnw wrapper |
| # Required ENV vars: | ||
| # ------------------ | ||
| # JAVA_HOME - location of a JDK home dir | ||
| # Apache Maven Wrapper startup batch script, version 3.3.4 |
There was a problem hiding this comment.
The comment on line 22 incorrectly refers to this as a "batch script" when this file is the Unix/Linux shell script wrapper. The comment should say "shell script" or "startup script" instead of "batch script". The batch script is in mvnw.cmd.
| # Apache Maven Wrapper startup batch script, version 3.3.4 | |
| # Apache Maven Wrapper startup shell script, version 3.3.4 |
