Skip to content

feat(java): add a java.shorthand_vendor setting#8134

Merged
jdx merged 3 commits intojdx:mainfrom
roele:issues/java-shorthand
Feb 13, 2026
Merged

feat(java): add a java.shorthand_vendor setting#8134
jdx merged 3 commits intojdx:mainfrom
roele:issues/java-shorthand

Conversation

@roele
Copy link
Contributor

@roele roele commented Feb 13, 2026

No description provided.

@gemini-code-assist
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@roele roele changed the title feat: add a java.shorthand_vendor setting feat(java): add a java.shorthand_vendor setting Feb 13, 2026
@roele roele force-pushed the issues/java-shorthand branch from 7335251 to 5c66242 Compare February 13, 2026 14:07
@roele roele marked this pull request as ready for review February 13, 2026 14:25
Copilot AI review requested due to automatic review settings February 13, 2026 14:25
Copy link
Contributor

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

This PR adds a new java.shorthand_vendor setting that allows users to configure which Java vendor is used when installing Java without an explicit vendor prefix (e.g., java@21 instead of java@openjdk-21). Previously, the vendor was hardcoded to "openjdk". This enhancement provides flexibility for users who prefer a different default vendor like Temurin.

Changes:

  • Adds a configurable java.shorthand_vendor setting with "openjdk" as the default value
  • Updates Java plugin code to use the setting instead of hardcoded "openjdk" strings
  • Includes comprehensive test coverage and documentation updates

Reviewed changes

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

Show a summary per file
File Description
src/plugins/core/java.rs Replaces hardcoded "openjdk" references with Settings::get().java.shorthand_vendor in metadata filtering and version conversion functions
settings.toml Defines the new java.shorthand_vendor setting with default value "openjdk"
schema/mise.json Adds JSON schema definition for the new java settings object and shorthand_vendor property
e2e/core/test_java_shorthand_vendor Adds end-to-end test verifying default openjdk behavior and custom temurin vendor configuration
docs/lang/java.md Updates documentation to explain the shorthand vendor behavior and reference the new setting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@roele roele force-pushed the issues/java-shorthand branch from 0c004ec to 4604b55 Compare February 13, 2026 22:15
@jdx jdx merged commit 9555810 into jdx:main Feb 13, 2026
34 checks passed
mise-en-dev added a commit that referenced this pull request Feb 14, 2026
### 🚀 Features

- **(java)** add a java.shorthand_vendor setting by @roele in
[#8134](#8134)

## 📦 Aqua Registry Updates

#### New Packages (4)

- [`IvanIsCoding/celq`](https://github.com/IvanIsCoding/celq)
- [`postfinance/topf`](https://github.com/postfinance/topf)
- [`runkids/skillshare`](https://github.com/runkids/skillshare)
- [`sandreas/tone`](https://github.com/sandreas/tone)
@roele roele deleted the issues/java-shorthand branch February 14, 2026 15:22
lucasew pushed a commit to lucasew/CONTRIB-mise that referenced this pull request Feb 18, 2026
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
lucasew pushed a commit to lucasew/CONTRIB-mise that referenced this pull request Feb 18, 2026
### 🚀 Features

- **(java)** add a java.shorthand_vendor setting by @roele in
[jdx#8134](jdx#8134)

## 📦 Aqua Registry Updates

#### New Packages (4)

- [`IvanIsCoding/celq`](https://github.com/IvanIsCoding/celq)
- [`postfinance/topf`](https://github.com/postfinance/topf)
- [`runkids/skillshare`](https://github.com/runkids/skillshare)
- [`sandreas/tone`](https://github.com/sandreas/tone)
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.

3 participants