Skip to content

SONARJAVA-6254 Use mise to install mvn in PrepareNextIteration.yml#5571

Merged
tomasz-tylenda-sonarsource merged 4 commits intomasterfrom
tt/prepare-next-iteration-workflow
Apr 15, 2026
Merged

SONARJAVA-6254 Use mise to install mvn in PrepareNextIteration.yml#5571
tomasz-tylenda-sonarsource merged 4 commits intomasterfrom
tt/prepare-next-iteration-workflow

Conversation

@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor

@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource commented Apr 14, 2026

Recent migration from GH runners to sonar runners broke the workflow, because our runners do not have mvn.

@sonar-review-alpha
Copy link
Copy Markdown
Contributor

sonar-review-alpha bot commented Apr 14, 2026

Summary

This PR adds the mise-action GitHub Actions step to the PrepareNextIteration workflow to install Maven. The sonar runners used by the workflow don't have mvn pre-installed (unlike the previous GitHub-hosted runners), so this step ensures Maven is available before the version normalization step runs.

What reviewers should know

What changed:
The only modification is in .github/workflows/PrepareNextIteration.yml — a single mise-action step was added immediately after checkout, specifying mise v2025.7.12.

What to verify:

  • Confirm that mise version 2025.7.12 is the correct/stable version for your use case
  • The step runs before any Maven-dependent steps, so the ordering is correct
  • If you're unfamiliar with mise, note that it's a polyglot version manager (like nvm, pyenv, etc.) — reviewers should check that your team is comfortable with this as a dependency

No breaking changes — this is a pure addition that fixes a missing dependency issue on the new runners.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

Copy link
Copy Markdown
Contributor

@sonar-review-alpha sonar-review-alpha bot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

Clean, minimal fix. The mise.toml at the repo root declares maven = "3.9", so mise will install it when the action runs — the Update Version Number step's mvn call will work correctly. The pinned commit hash and mise version match what's used across all jobs in build.yml, keeping things consistent.

🗣️ Give feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the fix. Could we update to a greater version of mise-action?

Comment thread .github/workflows/PrepareNextIteration.yml Outdated
Co-authored-by: Dorian Burihabwa <75226315+dorian-burihabwa-sonarsource@users.noreply.github.com>
github-actions bot and others added 2 commits April 15, 2026 10:16
Co-authored-by: tomasz-tylenda-sonarsource <tomasz-tylenda-sonarsource@users.noreply.github.com>
@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource changed the title Use mise to install mvn in PrepareNextIteration.yml SONARJAVA-6254 Use mise to install mvn in PrepareNextIteration.yml Apr 15, 2026
@sonarqube-next
Copy link
Copy Markdown

@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource merged commit 2ed1922 into master Apr 15, 2026
13 of 14 checks passed
@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource deleted the tt/prepare-next-iteration-workflow branch April 15, 2026 09:00
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