feat(java): add a java.shorthand_vendor setting#8134
Merged
Conversation
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
7335251 to
5c66242
Compare
Contributor
There was a problem hiding this comment.
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_vendorsetting 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.
0c004ec to
4604b55
Compare
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)
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)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.