Fix upgrade notice for JDK 20 users.

graalvm · Sep 20, 2023 · 9ace57c · 9ace57c
1 parent 0a27862
commit 9ace57c
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 30 deletions.
diff --git a/dist/main/index.js b/dist/main/index.js
diff --git a/src/features/check-for-updates.ts b/src/features/check-for-updates.ts
@@ -1,9 +1,9 @@
 import * as core from '@actions/core'
 
-export async function checkForUpdates(
+export function checkForUpdates(
   graalVMVersion: string,
   javaVersion: string
-): Promise<void> {
+): 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'

diff --git a/src/main.ts b/src/main.ts
@@ -43,6 +43,13 @@ async function run(): Promise<void> {
       distribution.length > 0 || graalVMVersion.length == 0
     let graalVMHome
     if (isGraalVMforJDK17OrLater) {
+      if (
+        enableCheckForUpdates &&
+        (distribution === c.DISTRIBUTION_GRAALVM ||
+          distribution === c.DISTRIBUTION_GRAALVM_COMMUNITY)
+      ) {
+        checkForUpdates(graalVMVersion, javaVersion)
+      }
       switch (distribution) {
         case c.DISTRIBUTION_GRAALVM:
           graalVMHome = await graalvm.setUpGraalVMJDK(javaVersion)
@@ -117,7 +124,7 @@ async function run(): Promise<void> {
             graalVMHome = await setUpMandrel(graalVMVersion, javaVersion)
           } else {
             if (enableCheckForUpdates) {
-              await checkForUpdates(graalVMVersion, javaVersion)
+              checkForUpdates(graalVMVersion, javaVersion)
             }
             graalVMHome = await graalvm.setUpGraalVMRelease(
               gdsToken,