Skip to content

Commit

Permalink
chore: add support for Node 15
Browse files Browse the repository at this point in the history
  • Loading branch information
nolanmar511 committed Dec 2, 2020
1 parent 01e0ac1 commit 7b5784a
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node: [ 10, 12, 14]
node: [ 10, 12, 14, 15]
steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
Expand Down
14 changes: 14 additions & 0 deletions system-test/Dockerfile.node15-alpine
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
FROM golang:1.15-alpine as builder
RUN apk add --no-cache git
WORKDIR /root/
RUN go get github.com/google/pprof


FROM node:15-alpine

ARG ADDITIONAL_PACKAGES

RUN apk add --no-cache bash $ADDITIONAL_PACKAGES
WORKDIR /root/
COPY --from=builder /go/bin/pprof /bin
RUN chmod a+x /bin/pprof
4 changes: 2 additions & 2 deletions system-test/system_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ if [[ "$RUN_ONLY_V8_CANARY_TEST" == "true" ]]; then
NVM_NODEJS_ORG_MIRROR="https://nodejs.org/download/v8-canary"
NODE_VERSIONS=(node)
else
NODE_VERSIONS=(10 12 14)
NODE_VERSIONS=(10 12 14 15)
fi

for i in ${NODE_VERSIONS[@]}; do
Expand All @@ -33,7 +33,7 @@ for i in ${NODE_VERSIONS[@]}; do

# Test support for accurate line numbers with node versions supporting this
# feature.
if [ "$i" != "10" ] && [ "$i" != "11" ]; then
if [ "$i" != "10" ]; then
docker run -v $PWD/..:/src -e BINARY_HOST="$BINARY_HOST" \
-e VERIFY_TIME_LINE_NUMBERS="true" node$i-linux \
/src/system-test/test.sh
Expand Down
3 changes: 3 additions & 0 deletions system-test/test.sh
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
#!/bin/bash

trap "cd $(dirname $0)/.. && npm run clean" EXIT
trap "echo '** TEST FAILED **'" ERR

. $(dirname $0)/../tools/retry.sh
Expand All @@ -22,6 +23,8 @@ cd $(dirname $0)/..

NODEDIR=$(dirname $(dirname $(which node)))

chmod 777 -R /root

# TODO: Remove when a new version of nan (current version 2.12.1) is released.
# For v8-canary tests, we need to use the version of NAN on github, which
# contains unreleased fixes that allow the native component to be compiled
Expand Down
2 changes: 1 addition & 1 deletion tools/build/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ mkdir -p "$ARTIFACTS_OUT"

npm install --quiet

for version in 10.0.0 12.0.0 14.0.0
for version in 10.0.0 12.0.0 14.0.0 15.0.0
do
./node_modules/.bin/node-pre-gyp configure rebuild package \
--target=$version --target_arch="x64"
Expand Down

0 comments on commit 7b5784a

Please sign in to comment.