fix: core.getInput returns string type (#138)

* fix incorrect ssh input check * fix incorrect ssh input check * Fixed true check * Try running in node 16 container * Try running in node 16 container * back to ubuntu-latest * node setup
wishabi · Oct 2, 2023 · af2fa89 · af2fa89
1 parent e2fd767
commit af2fa89
Show file tree

Hide file tree

Showing 8 changed files with 7,079 additions and 1,220 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -11,15 +11,18 @@ jobs:
   units:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
+    - uses: actions/setup-node@v3
+      with:
+        node-version: 16
     - run: npm ci
     - run: npm test
 
   # test action works running from the graph
   test:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - uses: ./
       with:
         milliseconds: 1000
diff --git a/cleanup.js b/cleanup.js
@@ -7,7 +7,7 @@ async function run() {
     const status = core.getInput("job_status")
     const enableSSH = core.getInput("enable_ssh")
     await slack();
-    if (enableSSH && (status === "failure" || core.getInput("ssh_always"))) {
+    if (enableSSH === "true" && (status === "failure" || core.getInput("ssh_always") === "true")) {
       await ssh();
     }
   } catch (error) {

diff --git a/dist/cleanup.js b/dist/cleanup.js
diff --git a/dist/cleanup.js.map b/dist/cleanup.js.map
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map