Merge pull request #611 from snyk/fix/node8-binaries

fix: make sure binaries are built on node8
snyk · Jun 27, 2019 · 8d77f32 · 8d77f32
2 parents f6cae17 + 3ecb38c
commit 8d77f32
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 16 deletions.
diff --git a/.releaserc b/.releaserc
@@ -2,14 +2,9 @@
   "prepare": [
     "@semantic-release/npm",
     {
-      "//": "build alpine binary inside an alpine container, or else the binary is broken",
+      "//": "build the alpine, macos, linux and windows binaries",
       "path": "@semantic-release/exec",
-      "cmd": "docker run --rm -i -v `pwd`:/snyk node:alpine sh /snyk/scripts/alpine-build.sh"
-    },
-    {
-      "//": "build the regular macos, linux and windows binaries",
-      "path": "@semantic-release/exec",
-      "cmd": "npm i -g pkg && pkg ."
+      "cmd": "npm i -g pkg && pkg . -t node8-alpine-x64,node8-linux-x64,node8-macos-x64,node8-win-x64"
     },
     {
       "//": "shasum all binaries",

diff --git a/scripts/alpine-build.sh b/scripts/alpine-build.sh