Skip to content

🐛 Fixing maven download and install for updated patch version#210

Merged
shawn-hurley merged 1 commit intokonveyor:mainfrom
shawn-hurley:bugfix/fix-maven-install
Mar 9, 2026
Merged

🐛 Fixing maven download and install for updated patch version#210
shawn-hurley merged 1 commit intokonveyor:mainfrom
shawn-hurley:bugfix/fix-maven-install

Conversation

@shawn-hurley
Copy link
Contributor

@shawn-hurley shawn-hurley commented Mar 9, 2026

Summary by CodeRabbit

  • Chores
    • Updated Maven toolchain from version 3.9.12 to 3.9.13, including updated download URLs and build environment configuration paths for the new version.

Signed-off-by: Shawn Hurley <shawn@hurley.page>
@coderabbitai
Copy link

coderabbitai bot commented Mar 9, 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: a513e2eb-7236-445b-a0ae-3ffd53b24d1c

📥 Commits

Reviewing files that changed from the base of the PR and between 0c44b5e and 3a9b9ee.

📒 Files selected for processing (2)
  • Dockerfile
  • Dockerfile.test

📝 Walkthrough

Walkthrough

Maven toolchain version is updated from 3.9.12 to 3.9.13 across two Dockerfile configurations. This includes updating download URLs, extraction paths, M2_HOME environment variables, and symlink references to point to the new Maven installation directory.

Changes

Cohort / File(s) Summary
Maven Version Bump
Dockerfile, Dockerfile.test
Updated Maven from 3.9.12 to 3.9.13, including download URLs, extraction directories, environment variable M2_HOME, and symlink targets.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Maven hops to thirteen, up from twelve before,
URLs and paths aligned forevermore,
The symlink points true to the new build way,
A rabbit's quick bump to brighten the day! 🔨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: updating Maven from version 3.9.12 to 3.9.13, which involves fixing the download and installation process.
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
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@shawn-hurley shawn-hurley merged commit c192e06 into konveyor:main Mar 9, 2026
15 checks passed
dymurray pushed a commit that referenced this pull request Mar 12, 2026
Signed-off-by: Shawn Hurley <shawn@hurley.page>
(cherry picked from commit c192e06)
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.

2 participants