Merge pull request #60 from crazy-max/node-16

Node 16 as default runtime
dagger · May 6, 2022 · 46924d8 · 46924d8
2 parents 4c4648c + cdfe7ba
commit 46924d8
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 6 deletions.
diff --git a/action.yml b/action.yml
@@ -31,6 +31,6 @@ inputs:
     required: false
 
 runs:
-  using: 'node12'
+  using: 'node16'
   main: 'dist/index.js'
   post: 'dist/index.js'
diff --git a/dev.Dockerfile b/dev.Dockerfile
@@ -1,6 +1,6 @@
 # syntax=docker/dockerfile:1
 
-ARG NODE_VERSION=12
+ARG NODE_VERSION=16
 
 FROM node:${NODE_VERSION}-alpine AS base
 RUN apk add --no-cache cpio findutils git

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -63,8 +63,11 @@ async function cleanup(): Promise<void> {
   if (!stateHelper.cleanup) {
     return;
   }
-  core.info(`Removing ${path.join(os.homedir(), '.config', 'dagger')}`);
-  fs.rmdirSync(path.join(os.homedir(), '.config', 'dagger'), {recursive: true});
+  const daggerHome = path.join(os.homedir(), '.config', 'dagger');
+  if (fs.existsSync(daggerHome)) {
+    core.info(`Removing ${daggerHome}`);
+    fs.rmSync(daggerHome, {recursive: true});
+  }
 }
 
 if (!stateHelper.IsPost) {