Skip to content

Commit

Permalink
Add upgrade notice for JDK 20 users.
Browse files Browse the repository at this point in the history
  • Loading branch information
fniephaus committed Sep 19, 2023
1 parent 44b2651 commit 2c392bc
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
6 changes: 5 additions & 1 deletion dist/main/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 7 additions & 1 deletion src/features/check-for-updates.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,17 @@ export async function checkForUpdates(
graalVMVersion: string,
javaVersion: string
): Promise<void> {
if (javaVersion === '20') {
core.notice(
'A new GraalVM release is available! Please consider upgrading to GraalVM for JDK 21: https://medium.com/p/ee01177dd12d'
)
return
}
if (
graalVMVersion.length > 0 &&
(javaVersion === '17' || javaVersion === '19')
) {
const recommendedJDK = javaVersion === '17' ? '17' : '20'
const recommendedJDK = javaVersion === '17' ? '17' : '21'
core.notice(
`A new GraalVM release is available! Please consider upgrading to GraalVM for JDK ${recommendedJDK}. Instructions: https://github.com/graalvm/setup-graalvm#migrating-from-graalvm-223-or-earlier-to-the-new-graalvm-for-jdk-17-and-later`
)
Expand Down

0 comments on commit 2c392bc

Please sign in to comment.