{{ message }}
dist: Support Tencent Kona JDK#672
Open
johnshajiang wants to merge 1 commit into
Open
Conversation
9871695 to
a8506e9
Compare
Contributor
Author
a8506e9 to
8549eb9
Compare
Contributor
Author
|
I just updated the commit again, and the file conflicts are resolved. |
41ce0ca to
109744b
Compare
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 13, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
109744b to
08f4b36
Compare
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 13, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
08f4b36 to
38b6474
Compare
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 13, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
38b6474 to
8b5e418
Compare
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 13, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
8b5e418 to
e003e7e
Compare
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 13, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
e003e7e to
6dd1e44
Compare
Contributor
Author
|
I already executed But |
johnshajiang
added a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Feb 14, 2025
Signed-off-by: John Jiang <johnsjiang@tencent.com>
Contributor
|
Linking the tracking issue: this PR implements #671 (Tencent Kona JDK support). Maintainers — adding |
brunoborges
pushed a commit
to johnshajiang/actions-setup-java
that referenced
this pull request
Jun 23, 2026
Signed-off-by: John Jiang <johnsjiang@tencent.com>
6dd1e44 to
59e214a
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds support for the Tencent Kona JDK distribution to actions/setup-java, including the installer implementation, documentation, and test/e2e coverage.
Changes:
- Add a new
KonaDistributionthat resolves Kona JDK downloads from the publishedkona-v1.jsonconfiguration. - Register the new
konadistribution in the distribution factory and bundleddistoutput. - Update docs, e2e matrix, and add unit tests + fixture manifest data for Kona.
Show a summary per file
| File | Description |
|---|---|
src/distributions/kona/models.ts |
Adds Kona release/config TypeScript models. |
src/distributions/kona/installer.ts |
Implements Kona distribution resolution and download/extract logic. |
src/distributions/distribution-factory.ts |
Registers kona as a supported distribution and wires installer creation. |
README.md |
Documents kona as a supported distribution and links to advanced usage. |
docs/advanced-usage.md |
Adds a “Tencent Kona” usage section and TOC entry. |
__tests__/distributors/kona-installer.test.ts |
Adds unit tests for Kona release selection and error cases. |
__tests__/data/kona.json |
Adds fixture manifest data used by the Kona unit tests. |
.github/workflows/e2e-versions.yml |
Adds kona to the e2e distribution matrix. |
dist/setup/index.js |
Updates the bundled action output to include the new Kona installer. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 8/9 changed files
- Comments generated: 5
Comment on lines
+63
to
+82
| const availableReleases = await this.getAvailableReleases(); | ||
| const releases = availableReleases | ||
| .filter(item => { | ||
| return isVersionSatisfies(version, item.version); | ||
| }) | ||
| .map(item => { | ||
| return { | ||
| version: item.version, | ||
| url: item.downloadUrl | ||
| } as JavaDownloadRelease; | ||
| }); | ||
|
|
||
| if (!releases.length) { | ||
| throw new Error( | ||
| `No Kona release for the specified version "${version}" on OS "${this.getOs()}" and arch "${this.getArch()}".` | ||
| ); | ||
| } | ||
|
|
||
| return releases[0]; | ||
| } |
Comment on lines
+4
to
+6
| import fs from 'fs'; | ||
| import path from 'path'; | ||
|
|
Comment on lines
+14
to
+18
| import { | ||
| extractJdkFile, | ||
| getDownloadArchiveExtension, | ||
| isVersionSatisfies | ||
| } from '../../util'; |
Comment on lines
+31
to
+36
| const javaArchivePath = await tc.downloadTool(javaRelease.url); | ||
|
|
||
| core.info(`Extracting Java archive...`); | ||
|
|
||
| const extension = getDownloadArchiveExtension(); | ||
| const extractedJavaPath = await extractJdkFile(javaArchivePath, extension); |
Comment on lines
+85
to
+104
| if (core.isDebug()) { | ||
| console.time('Retrieving available releases for Kona took'); // eslint-disable-line no-console | ||
| } | ||
|
|
||
| const releaseInfo = await this.fetchReleaseInfo(); | ||
| if (!releaseInfo) { | ||
| throw new Error(`Couldn't fetch Kona release information`); | ||
| } | ||
|
|
||
| const availableReleases = this.chooseReleases( | ||
| this.getOs(), | ||
| this.getArch(), | ||
| releaseInfo | ||
| ); | ||
|
|
||
| if (core.isDebug()) { | ||
| core.startGroup('Print information about available releases'); | ||
| core.debug(availableReleases.map(item => item.version).join(', ')); | ||
| core.endGroup(); | ||
| } |
Signed-off-by: John Jiang <johnsjiang@tencent.com>
59e214a to
39f01ea
Compare
Contributor
Author
brunoborges
approved these changes
Jun 24, 2026
3 tasks
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.

Description:
This PR adds Tencent Kona JDK into setup-java.
It supports the major versions 8, 11, 17 and 21.
The supported platforms are Linux x86_64/aarch64, macOS x86_64/aarch64 and Windows x86_64.
Currently, only the latest stable releases are supported, and only jdk (no jre) binaries are available.
A configuration maintains these releases.
Related issue:
#671
Check list: